Skip to content

34225 numbered company launcher - #879

Open
kialj876 wants to merge 7 commits into
bcgov:mainfrom
kialj876:34225-numbered-company-launcher
Open

34225 numbered company launcher#879
kialj876 wants to merge 7 commits into
bcgov:mainfrom
kialj876:34225-numbered-company-launcher

Conversation

@kialj876

@kialj876 kialj876 commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

Issue #: /bcgov/entity#34225

Description of changes:

  • enhancement to new incorporation flow for newly modernized corps (BC, ULC, CC)
    • gives option to user for colin or modern incorporation with help me choose modal for corps (except for BENs)
    • feature flagged
  • Amalgamation/Continuation In/Change of Name/Alteration/Restoration flows are unaffected
  • examples:

Only shows for new incorporation for BC/ULC/CC numbered companies:
numbered_company_launcher

new dialog when user clicks 'help me choose':
numbered_company_launcher_dialog

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

kialj876 added 3 commits July 24, 2026 16:22
… choose dialog

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
Signed-off-by: Kial Jinnah <kialj876@gmail.com>
Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@kialj876

This comment was marked as duplicate.

@kialj876
kialj876 requested a review from meawong July 27, 2026 14:44
@kialj876

Copy link
Copy Markdown
Collaborator Author

CI is passing locally - looking into issue with the PR run CI

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@kialj876
kialj876 force-pushed the 34225-numbered-company-launcher branch from d05124e to 73fec2f Compare July 27, 2026 19:00
Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@kialj876

This comment was marked as duplicate.

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@kialj876
kialj876 force-pushed the 34225-numbered-company-launcher branch from 0b0c4b7 to fe0ef65 Compare July 27, 2026 19:25
… (required since CI prevents post install script)

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
Comment thread app/package.json
"test": "vitest run",
"test:unit": "vitest run",
"test": "vue-demi-fix && vitest run",
"test:unit": "vue-demi-fix && vitest run",

@kialj876 kialj876 Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

required since SRE CI installs with '--ignore-scripts'

@kialj876

Copy link
Copy Markdown
Collaborator Author

/gcbrun

@kialj876

kialj876 commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator Author

/gcbrun

The preview build is broken because someone updated it to use pnpm 11.9 so its not an issue with this PR. I do not want to update it because it looks like there's in progress work to do with this going on someone fixed it, thanks

@mruff-aeq

Copy link
Copy Markdown
Collaborator

/gcbrun

3 similar comments
@mruff-aeq

Copy link
Copy Markdown
Collaborator

/gcbrun

@meawong

meawong commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

/gcbrun

@meawong

meawong commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

/gcbrun

@bcregistry-sre

Copy link
Copy Markdown
Collaborator

Temporary Url for review: https://namerequest-dev--pr-879-721dj2ic.web.app

@meawong meawong left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice! Tested in the preview link and LGTM

@stevenc987

stevenc987 commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

@kialj876 As discussed with David M, could you please hold off on merging to main until we decide when solr will be deployed to PROD?

@kialj876

Copy link
Copy Markdown
Collaborator Author

@kialj876 As discussed with David M, could you please hold off on merging to main until we decide when solr will be deployed to PROD?

yep no problem

@stevenc987

stevenc987 commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

@kialj876

Solr has been rolled back this morning so please keep holding this codes. If you want it to be merged to main earlier, please talk to David M.

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.

5 participants