-
Notifications
You must be signed in to change notification settings - Fork 45
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
24084 - Alteration Name/Type Change Logic #596
24084 - Alteration Name/Type Change Logic #596
Conversation
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
Signed-off-by: Qin <Arwen.Qin@gov.bc.ca>
/gcbrun |
Temporary Url for review: https://business-edit-dev--pr-596-b5vobzl5.web.app |
I tried changing business type to ULC, and checking the checkbox, but the Done button is still disabled. I should be able to change the company name to allow the business type change to proceed: either with a BC -> ULC NR, or by changing to numbered. I didn't try the former, and the latter doesn't give me the right numbered name (it shows "xxx BC Ltd" instead of "xxx Unlimited Liability Company". Finally, I saved by changes and reopened the alteration, and that didn't help. (Previously when I tested, saving and then reopening fixed some things.) Make sure you test:
|
By the current change, I fixed Named BC -> Named ULC, by entering the NR. However, Named BC -> Numbered ULC, by changing to numbered, will not work. It only works by filing 2 separate changes: So a Named BC will change to "xxx BC Ltd" in the first step, just to change a named to a numbered, but can't change its type at the same type. In summary: What do you think? |
Can you figure out why it doesn't work, or what's needed to fix it? I don't like needing 2 filings as that's twice the fees, but named BC → numbered ULC is possibly much less likely than named → named, so maybe it's OK. Or maybe it's OK for now and we can fix it further later. Please check with Mihai and Olga. |
The reason is the current functionality only allows change from a named to a numbered by appending "B.C. Ltd." / or "UNLIMITED LIABILITY COMPANY", just same with the current legal type. It's not linking to the type change function. I will check with Olga and Mihai |
I think you are on the right track: the name change needs to be linked to the type change. Question: should the user change name before type, or type before name, or it shouldn't matter? |
I am thinking about:
|
How does it work when you use a NR -- can you use a NR first and then change type, or does it have to be the other way around? I think option 2 makes the most sense. We have to make sure there is a note that says the name (/number) must be changed before they can click Done. |
When use a NR:
For NR to numbered, I think both scenario 1 and 2 can happen. For scenario 2, when the error message shows, we can add a note that says the name (/number) must be changed before they can click Done. |
Arwen, can you please summarize all possibly scenarios/flows along with what works and what doesn't? It will help me and when explaining to Mihai/Olga. My goals are: clarity, consistency and correctness :) |
Hi Severin, in summary:
Possible flows after update:
|
Olga said |
OK, let's go with this. Please copy-paste your awesome summary from above into the ticket. Thanks! |
@severinbeauvais shall we merge this PR now? |
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.
LGTM
Issue #: /bcgov/entity#24084
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).