-
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
Add "Definitions of ZAP Updates" page to ZAP Search #1535
Comments
@dhochbaum-dcp That link should link to |
Hi @jessicashanshanhuang, can you please take a look at https://deploy-preview-1537--labs-zap.netlify.app/statuses and let me know if the design is ok? (The "subscribe" link is in the code, but since the page does not yet exist, the word linking it is not displaying) |
Hey @dhochbaum-dcp, looks great overall! I just have some minor suggestions with the spacing: The spacing between the header and the "Definition of Zap Updates" should be increased, about 4rem. |
@jessicashanshanhuang I've made those updates, please re-check and let me know if you approve. |
@dhochbaum-dcp Looks good to go 🫡 |
Add new "Definition of ZAP Updates" page to ZAP Search.
Acceptance Criteria:
/statuses
/subscribe
(not implemented)SHOW_ALERTS
feature flag istrue
.Out of scope
@TylerMatteo to follow up with @jessicashanshanhuang about mobile designs
The text was updated successfully, but these errors were encountered: