-
Notifications
You must be signed in to change notification settings - Fork 3
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
About Security Content #2127
About Security Content #2127
Conversation
@heyitsmebev all links that leave the Notify.gov site need an external link icon added Example:
|
Security is a subpage to the About Notify section. It is navigated to after a user selects to view the About Notify section. @CathyBeil and @jonathanbobel, correct, yes? Note: It is in front of the login, but i understand that's a different PR. |
@dmvancura i revised the menu hierarchy in this PR. Security is indeed a subnav |
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.
LGTM
anything else needed on 2127? github is a bit confusing and i can't tell by looking through this, so let me know if i need to help with anything else @jonathanbobel |
As part of the new About Notify content section, this PR is the implementation for "Security"
Definition of Done
Figma:
https://www.figma.com/proto/aAWhlPzOP6kfVFnCCAfSZZ/notify.gov?page-id=2339%3A3817&node-id=5392-875&node-type=frame&viewport=2870%2C-701%2C0.48&t=nfkxOXnOCIFQB1yK-1&scaling=min-zoom&content-scaling=fixed&starting-point-node-id=5392%3A875
Content:
https://docs.google.com/document/d/1lP8I0Jxu0B26cmTGBQcoHCbfSvKKHQBE6gzTL9HUheE/edit?tab=t.0#heading=h.icyigvoln8tg