Skip to content
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

25294 - Submit NoW for Bootstrap Filing #712

Merged
merged 9 commits into from
Jan 17, 2025

Conversation

ArwenQin
Copy link
Collaborator

Issue #: /bcgov/entity#25294

Description of changes:

  • Fixed the issue: can't submit NoW for Bootstrap Filings
    Reason: Bootstrap filings don't have those 3 properties set up - founding date, legal name and Registered Office email.
  • Now stores the 3 properties in a bootstrap item's data fields in Filing UI
  • Still can't Save/ Save and Resume, probably a bug in Legal API

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).

src/App.vue Show resolved Hide resolved
src/App.vue Outdated
@@ -624,6 +625,12 @@ export default class App extends Mixins(
const filing = response?.filing
const filingName = filing.header?.name as FilingTypes
const status = filing.header.status as FilingStatus
const foundingDate = filing.header?.effectiveDate // use the FE date as the founding date
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the FE date as the founding date

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great comment, thanks!

src/App.vue Show resolved Hide resolved
@ArwenQin ArwenQin self-assigned this Jan 15, 2025
@ArwenQin ArwenQin requested a review from eason-pan-bc January 15, 2025 22:59
@ArwenQin
Copy link
Collaborator Author

ArwenQin commented Jan 15, 2025

@ArwenQin
Copy link
Collaborator Author

Still can't Save/ Save and Resume, probably a bug in Legal API - need a new ticket
click "Save" or "Save and Resume" for the first time - no error, but nothing is saved
click it twice, show the no filing found error:
image

We experienced the similar issue before, probably an issue in Legal API

@ArwenQin

This comment was marked as resolved.

// special case to identify numbered amalgamations
if (filingName === FilingTypes.AMALGAMATION_APPLICATION) {
this.setLegalName(nameRequest.legalName || 'Numbered Amalgamated Company')
} else { this.setLegalName(nameRequest.legalName || GetCorpNumberedDescription(this.getLegalType)) }

Copy link
Collaborator Author

@ArwenQin ArwenQin Jan 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Special case for numbered amalgamations, same as current set up in businessStore.ts:
image

other bootstraps, set legal name to numbered description, instead of null

Amalgamation (same as current display)
image

@bcgov bcgov deleted a comment from bcregistry-sre Jan 15, 2025
src/App.vue Show resolved Hide resolved
@bcgov bcgov deleted a comment from bcregistry-sre Jan 16, 2025
@bcgov bcgov deleted a comment from bcregistry-sre Jan 16, 2025
@bcgov bcgov deleted a comment from bcregistry-sre Jan 16, 2025
@ArwenQin
Copy link
Collaborator Author

/gcbrun

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
10.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@bcgov bcgov deleted a comment from bcregistry-sre Jan 16, 2025
@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://business-filings-dev--pr-712-mw95fpzl.web.app

@ArwenQin ArwenQin merged commit 5cf66ef into bcgov:main Jan 17, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants