Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds stakeholders section pages #178

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Adds stakeholders section pages #178

merged 1 commit into from
Jul 11, 2023

Conversation

brentryanjohnson
Copy link
Contributor

  • Adds three pages in stakeholders section

Warning
Previews won't work, since we don't yet have a [...slug].astro page for this section (or for the how-it-works section)

Note
One of page titles in Figma ("Agency/Organization Program Coordinators (A/OPCs) Responsibilities" differs from that of the corresponding google doc ("Program Coordinator Responsibilities"). Also in Figma, the side nav title ("A/OPC Responsibilities") is different than the page title. To my knowledge, the engineers pull the page title for use in the side nav, and I don't think we currently have a way to use a side nav title that's different from the corresponding page title.

Note
So far, we've assumed a landing page for each section (index.md). This page uses "Key Players" for the landing page, which would render at the path /stakeholders/ instead of /stakeholders/key-players/

@mark-meyer
Copy link
Contributor

@brentryanjohnson do you want to merge this in, or wait until we do some dev work to make it live?

@brentryanjohnson
Copy link
Contributor Author

brentryanjohnson commented Jul 11, 2023

@mark-meyer I think we can merge it so the branch doesn't get stale. Should we add issues for the Astro pages we need?

@mark-meyer mark-meyer merged commit 2a47bc1 into main Jul 11, 2023
@brentryanjohnson brentryanjohnson deleted the rj-section-stakeholders branch September 11, 2023 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants