-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Fix 24953 - Update NoW Per New Designs #709
Conversation
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
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.
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
Quality Gate passedIssues Measures |
/gcbrun |
Temporary Url for review: https://business-filings-dev--pr-709-473wrfuv.web.app |
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.
Looks good with the fix!
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 😃
Thanks reviewers. I want to confirm with Jacqueline tmr and then merge the changes. |
Issue #: /bcgov/entity#24953
Description of changes:
Jacqueline's UX assurance notes are here:
https://www.figma.com/design/CLqwqGQYARaSrnzLwh6w8Q/Notice-of-Withdrawal?node-id=700-16551&p=f&focus-id=2462-239471&m=dev
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the business-filings-ui license (Apache 2.0).