-
Notifications
You must be signed in to change notification settings - Fork 66
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
15769 - Manage a B.C. Business modal view for Corporation #2468
15769 - Manage a B.C. Business modal view for Corporation #2468
Conversation
auth-web/src/components/auth/manage-business/ManageBusinessDialog.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/ManageBusinessDialog.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/ManageBusinessDialog.vue
Outdated
Show resolved
Hide resolved
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.
Please nudge package.json and package lock as well - I will be moving it up a minor version when we merge to main though
/gcbrun |
Temporary Url for review: https://bcregistry-account-dev--pr-2468-ozcq53tu.web.app |
/gcbrun |
Temporary Url for review: https://bcregistry-account-dev--pr-2468-ozcq53tu.web.app |
auth-web/src/components/auth/manage-business/EntityManagement.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/AffiliatedEntityTable.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/EntityManagement.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/ManageBusinessDialog.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/ManageBusinessDialog.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/AffiliatedEntityTable.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/EntityManagement.vue
Outdated
Show resolved
Hide resolved
auth-web/src/components/auth/manage-business/EntityManagement.vue
Outdated
Show resolved
Hide resolved
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.
There are some concerns to address before this gets merged
Kudos, SonarCloud Quality Gate passed! |
/gcbrun |
41eed5b
into
bcgov:feature-business-registry-dashboard-updates
Temporary Url for review: https://bcregistry-account-dev--pr-2468-ozcq53tu.web.app |
Issue #:15769
bcgov/entity#15769
Additions and changes for managing a B.C. business modal for corporations. Visual fixes to related modals, and copy updates.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the sbc-auth license (Apache 2.0).