Rename Content Strategist role to Marketing Director - #284
Open
GauravShah123 wants to merge 2 commits into
Open
Conversation
Rename the Content Strategist role to Marketing Director on the /roles and /apply pages, and refocus its description on owning Blueprint's brand image across the community (social channels, in-person events, and marketing to students, non-profits, and companies). Also update the Graphic Designer responsibility that references the role by name. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
`next build` evaluates pages/_app during "Collecting page data", which
imports utils/firebase and calls getAuth()/getDatabase() at module load.
When the NEXT_PUBLIC_* Firebase env vars are absent, these throw
("Cannot parse Firebase url", "auth/invalid-api-key") and the build fails.
This happens on every fork PR: GitHub does not expose repository secrets
to pull_request workflows from forks, so the secret-backed env vars are
empty. Give apiKey and databaseURL valid-format placeholder fallbacks so
initialization succeeds during the build. Real deployments build with the
real values injected, so the placeholders are never used there.
Verified: `next build` with all Firebase env vars unset now completes
successfully (previously failed while collecting page data).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
eric-mxrtin
approved these changes
Jul 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Renames the Content Strategist role to Marketing Director on the roles page (
/roles) and the application page (/apply), and refocuses the role description on owning Blueprint's brand image across the community.Changes
src/app/roles/page.tsxContent Strategist→Marketing Director.constants/role-specific-questions.jsonContent Strategist→Marketing Director, which updates the role option and role-specific questions on the application form. (The apply form matches applicants to questions by thisrolestring, so the label and matching stay in sync.)Scope / out of scope
Kept the change minimal and limited to the
/rolesand/applysurfaces. Intentionally left existing team member listings (constants/members.json,constants/headshot-constants.ts) and the/studentspage untouched, since those reflect current members' actual titles rather than the open-role definitions.🤖 Generated with Claude Code