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

Create UI: Resulting business name not updated if primary/holding name was changed in the background #19756

Closed
riyazuddinsyed opened this issue Feb 8, 2024 · 5 comments
Assignees
Labels
bug Something isn't working Entities - Olga A label to filter on the tickets for the Entities based team that Olga is PO for. Priority2 UX Assurance

Comments

@riyazuddinsyed
Copy link
Collaborator

riyazuddinsyed commented Feb 8, 2024

Original title: "Resulting business name doesn't align with the Notice of Articles of the holding name when an Alteration is filed."

Describe the bug in the current situation
When the amalgamation is in progress or say in draft state and if in case the user makes a name change using an Alteration for say Numbered to Named then in the Amalgamation filing upon resume draft the Amalgamating business table shows the name change while as the Resulting business name doesnt chnage the name as per the NOA.

Steps to reproduce the behavior:

  1. Go to 'https://dev.bcregistry.gov.bc.ca/?accountid=2079'
  2. Login with BCREGA001/98900001
  3. Search the Draft Amalgamation filing for the company 0871305 B.C. LTD.
  4. Now under the primary business notice the name in the Amalgamating table and the name in the Resulting business name section.

Screenshots/ Visual Reference/ Source

Screen Shot 2024-02-08 at 12.45.06 AM.png

Screen Shot 2024-02-08 at 12.48.39 AM.png

@riyazuddinsyed riyazuddinsyed added bug Something isn't working Entities - Olga A label to filter on the tickets for the Entities based team that Olga is PO for. labels Feb 8, 2024
@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Feb 8, 2024

deleted obsolete comment due to new comment below

@severinbeauvais severinbeauvais changed the title Resulting business name doesn't align with the Notice of Articles of the holding name when an Alteration is filed. Output: Resulting business name doesn't align with the Notice of Articles of the holding name when an Alteration is filed. Feb 14, 2024
@severinbeauvais severinbeauvais changed the title Output: Resulting business name doesn't align with the Notice of Articles of the holding name when an Alteration is filed. Resulting business name not updated if primary/holding name was changed in the background Feb 15, 2024
@severinbeauvais severinbeauvais self-assigned this Feb 15, 2024
@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Feb 15, 2024

There are 2 cases to handle:

  1. The holding/primary business name (or legal type!) has changed and the user reloads the draft amalgamation. To handle this case, when the draft is reloaded then we need to reload the resulting business name.

  2. The holding/primary business name (or legal type!) has changed and the user tries to submit the still-open amalgamation. To handle this case, when the filing is submitted then we need to reload the resulting business name.

(Note that the Filer ignores the legal name in the filing JSON and looks up the legal name of the holding/primary name anyway.)

There is already an "update prepopulated data" method that is called in the 2 cases above, but a small code tweak is needed. I will do that in this ticket and verify all cases.

cc: @OlgaPotiagalova

@severinbeauvais severinbeauvais changed the title Resulting business name not updated if primary/holding name was changed in the background Create UI: Resulting business name not updated if primary/holding name was changed in the background Feb 16, 2024
@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Feb 16, 2024

Test Notes

I tested the following on a horizontal and vertical amalgamation:

  1. open 2 copies of the amalgamation
  2. in the background, change the name of the holding/primary business
  3. reopen (resume/refresh) one copy of the amalgamation, and verify that the holding/primary business' name has changed and also the amalgamation's resulting business name
  4. click File and Pay to submit the other copy of the amalgamation, and verify that the holding/primary business' name has changed and also the amalgamation's resulting business name (if you want, you can leave the amalgamation incomplete so that File and Pay refreshes the data but the filing is not actually submitted)

I also tested, in a regular amalgamation, that a TING's name is updated in the business table (if it was changed in the background) when the user clicks File and Pay to submit the amalgamation.

@severinbeauvais
Copy link
Collaborator

Test Notes (cont)

I also tested that changing the type of the holding/primary business in the background is caught when the user clicks File and Pay to submit the filing, as follows:

  1. create a short-form amalgamation from a CCC business (which makes the amalgamation a CCC and sets the business as the holding/primary)
  2. add a Benefit Company and mark this BEN as the new holding/primary); this causes a table error because a CCC must be the holding/primary
  3. fill out the rest of the amalgamation and try to submit it -- observe the table error
  4. in the background, change the BEN to a CCC
  5. come back to the amalgamation and try to submit it -- observe that submission now succeeds because the table error was resolved because the holding/primary, which was previously a BEN, is now a CCC

@yuisotozaki
Copy link

Good catch, this.
Ready for QA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Entities - Olga A label to filter on the tickets for the Entities based team that Olga is PO for. Priority2 UX Assurance
Projects
None yet
Development

No branches or pull requests

5 participants