-
Notifications
You must be signed in to change notification settings - Fork 70
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
Document Record Service - Cont. In Integration #3242
base: feature-drs-integration
Are you sure you want to change the base?
Document Record Service - Cont. In Integration #3242
Conversation
auth-web/src/components/auth/staff/continuation-application/HomeJurisdictionInformation.vue
Outdated
Show resolved
Hide resolved
|
@@ -314,13 +315,23 @@ export default defineComponent({ | |||
if (!documentKey || !documentName) return // safety check | |||
|
|||
state.isDownloading = true | |||
await BusinessService.downloadDocument(documentKey, documentName).catch(error => { | |||
try { | |||
if (DRS_ID_PATTERN.test(documentKey)) { |
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.
do we need a feature flag for this, or can it be pushed out to prod next week if needed etc?
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.
@seeker25
I changed the base branch since we are holding the updates on feature branch on create-ui
repo.
another thing to note, product/payment changes will probably have to sit on TEST for a while - just a heads up |
Ok keep in mind anytime you merge into
|
Issue #:
bcgov/entity#24640
Description of changes:
My name is Brandon Galli from Assets team.
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).