-
Notifications
You must be signed in to change notification settings - Fork 67
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
Claims and applications page #3504
Conversation
minor tweaks
tweak
initial edits after discussing with Danielle and Laura
adding life insurance
minor copy tweak
added language for "claimant"
added sponsor
adding changes discussed with LW and DT
Update claims-and-applications.md
Hi @humancompanion-usds, this PR is ready for the content style guide. It's the special topic page about claims and applications that we showed you on a previous design/content sync. Do you mind doing a final review and merging when you have a minute? @DanielleThierryUSDSVA and @jstrothman both reviewed and approved it. Thank you!! CC @laurwill |
Hi @danbrady, I'm pinging you since Matthew's out at the offsite. Any chance you could review and merge this PR for the content style guide? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this is a new page, we'll have to add it into the left side navigation so it's accessible. Also, we would usually update the "In this section" part on the page level above (in this case, "Specific topics and programs") with a link to this new page. Are you able to do that?
@@ -0,0 +1,59 @@ | |||
--- | |||
layout: content-style-guide | |||
permalink: /content-style-guide/specific-topics-and-programs/claims |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see the page was renamed to "claims and applications", so we likely want to update the permalink as well.
Adding subpage
changing permalink to match title
Thank you for catching that, @danbrady!! I adjusted the permalink and added it as a sub-page on the "Specific topics and programs" index page. But I don't see where to link to it or how to add it to the nav. Could you check again and see if there are more changes I need to make? |
@megzehn My mistake, I forgot that the left nav updates automatically based on what's in the "sub-pages" front matter. I can see it now after you added it to that section. Looks good. Approved! |
Creating a PR for a new special topics page: claims and applications