-
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
fix(ui): Appendix K Form, Instructions, and Modal Design Updates #936
Conversation
…clude 'Appendix K'
…ndix k in the titles
Coverage Report
File Coverage
|
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.
Left some nits, but do you think a function that formats 1915(c)
to 1915(c) Appendix K
would save some repetition?
react-app/src/features/forms/post-submission/respond-to-rai/index.tsx
Outdated
Show resolved
Hide resolved
react-app/src/features/forms/post-submission/withdraw-package/index.tsx
Outdated
Show resolved
Hide resolved
react-app/src/features/forms/post-submission/withdraw-package/index.tsx
Outdated
Show resolved
Hide resolved
@asharonbaltazar Updated to address your feedback. For you question on how repetitive that format will be, I don't know. Based on your knowledge do you think it's something we should abstract into a utility or something like that? |
🎉 This PR is included in version 1.5.0-val.98 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎫 Linked Ticket
Closes OY2-31581
💬 Description / Notes