Skip to content

35070 - CORP correction filings conditionally include NOA report. - #4843

Open
doug-lovett wants to merge 3 commits into
bcgov:mainfrom
doug-lovett:35070-corp-correction-noa
Open

doug-lovett wants to merge 3 commits into
bcgov:mainfrom
doug-lovett:35070-corp-correction-noa

Conversation

@doug-lovett

@doug-lovett doug-lovett commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Issue #: /bcgov/entity#35070

Description of changes:
A correction filing conditionally produces a NOA report if any of the following are updated:

  • name
  • office address
  • directors
  • share structure
  • corp type
  • continuation in jurisdiction
  • Update the business filer to set filing.meta_data["correction"]["hasNoa"] as a boolean based on what was corrected.
  • Update the business emailer to include a NOA report with correction filings based on the filing.meta_data["correction"]["hasNoa"] value.
  • Update the business/legal api to include the NOA report in the ledger based on the filing.meta_data["correction"]["hasNoa"] value.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the lear license (Apache 2.0).

Signed-off-by: Doug Lovett <doug@diamante.ca>
@doug-lovett doug-lovett self-assigned this Oct 7, 2026
Signed-off-by: Doug Lovett <doug@diamante.ca>
Signed-off-by: Doug Lovett <doug@diamante.ca>
@sonarqubecloud

sonarqubecloud Bot commented Oct 7, 2026

Copy link
Copy Markdown

@doug-lovett

doug-lovett commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator Author

@kialj876 Hi Kial, for some reason I can't assign reviewers - can you please assign yourself and add some others? Thanks. Please ignore, now I can.

@kialj876 kialj876 changed the title CORP correction filings conditionally include NOA report. 35070 - CORP correction filings conditionally include NOA report. Oct 7, 2026

This branch has not been deployed

No deployments
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.

2 participants