-
Notifications
You must be signed in to change notification settings - Fork 51
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
21948 - Change Continuation Application #679
21948 - Change Continuation Application #679
Conversation
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
Temporary Url for review: https://business-filings-dev--pr-679-2vtz07q6.web.app |
src/components/Dashboard/FilingHistoryList/bodies/FutureEffectivePending.vue
Outdated
Show resolved
Hide resolved
src/components/Dashboard/FilingHistoryList/bodies/FutureEffective.vue
Outdated
Show resolved
Hide resolved
Arwen, please revert the change in NameRequestInfo.vue (per Andy), and then attach some screenshots of what the Future Effective and Future Effective Pending filings look like. Did you confirm that the default values are not being used? Thanks. |
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
/gcbrun |
Hi Severin, I just updated the UIs per design of Andy, please see below. |
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. Ask Ketaki if she has time to review. If not, go ahead and merge.
@@ -52,7 +53,7 @@ export default class FutureEffective extends Vue { | |||
return { | |||
subtitle: 'Future Effective Incorporation Date', | |||
filingLabel: 'incorporation', | |||
filingTitle: 'Incorporation Application' | |||
filingTitle: FilingNames.INCORPORATION_APPLICATION |
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.
Line 56, 70, 77: used enum FilingNames for filingTitle
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.
Sorry for the delay. Looks good Arwen!
Issue #: /bcgov/entity#21948
Description of changes:
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).