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

Legal API: fix alteration of company type issues #24041

Closed
10 of 14 tasks
severinbeauvais opened this issue Oct 25, 2024 · 9 comments
Closed
10 of 14 tasks

Legal API: fix alteration of company type issues #24041

severinbeauvais opened this issue Oct 25, 2024 · 9 comments
Assignees
Labels
bug Something isn't working Entities - Lorrie A label to filter on the tickets for the Entities based team that David is PO for. Priority2

Comments

@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Oct 25, 2024

See results from #23941.

Steps to Reproduce:

  1. Open a BC numbered business (impacting regular and continued-in)
  2. Start a business type alteration.
  3. Select BC Community Contribution Company from dropdown (i.e. try to alter BC -> CC or C -> CCC).
  4. Proceed with regular filing workflow.
  5. Click File and Pay button.
  6. Unable to save Filing error pop-up appears.

Edit: Ensure the following works:

  • BC -> BEN
  • BC -> ULC
  • BC -> CC
  • BEN -> BC
  • BEN -> CC (updated legal api to allow)
  • ULC -> BC
  • ULC -> BEN
  • C -> CBEN
  • C -> CUL
  • C -> CCC
  • CBEN -> C
  • CBEN -> CCC (updated legal api to allow)
  • CUL -> C
  • CUL -> CBEN

Also, ensure the above codes work for C, CBEN, CCC and CUL business types. Edit: Updated above codes to list every allowable alteration in lear. If any are missing let me know.

@severinbeauvais severinbeauvais 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 Oct 25, 2024
@meawong
Copy link
Collaborator

meawong commented Oct 29, 2024

Steps to Reproduce:

  1. Open a BC numbered business (impacting regular and continued-in)
  2. Start a business type alteration.
  3. Select BC Community Contribution Company from dropdown (i.e. try to alter BC -> CC or C -> CCC).
  4. Proceed with regular filing workflow.
  5. Click File and Pay button.
  6. Unable to save Filing error pop-up appears.

image.png

image.png

image.png

Alteration attempted on BC0883647 from screenshots.

@davemck513 davemck513 added Entities - Lorrie A label to filter on the tickets for the Entities based team that David is PO for. and removed Entities - Olga A label to filter on the tickets for the Entities based team that Olga is PO for. labels Oct 30, 2024
@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Nov 1, 2024

Here are the updated requirements.

  1. BC -> BEN
  2. BC -> ULC
  3. BC -> CC
  4. BEN -> BC
  5. BEN -> CC
  6. ULC -> BC
  7. ULC -> BEN

(and all the continued-in equivalents)

@vysakh-menon-aot
Copy link
Collaborator

@OlgaPotiagalova @vikas-gov Since we are enabling BEN->CC alteration, we need to document it somewhere that these businesses will be out of sync with COLIN until we introduce an equivalent filing type in COLIN

@severinbeauvais
Copy link
Collaborator Author

@OlgaPotiagalova @vikas-gov Since we are enabling BEN->CC alteration, we need to document it somewhere that these businesses will be out of sync with COLIN until we introduce an equivalent filing type in COLIN

How about another sync ticket?

@vysakh-menon-aot
Copy link
Collaborator

@OlgaPotiagalova @vikas-gov Since we are enabling BEN->CC alteration, we need to document it somewhere that these businesses will be out of sync with COLIN until we introduce an equivalent filing type in COLIN

How about another sync ticket?

That's needed for sure. I was saying either we need to disable this using FF in PROD (unless we are ok with out of sync data)

@severinbeauvais
Copy link
Collaborator Author

I was saying either we need to disable this using FF in PROD (unless we are ok with out of sync data)

If we don't expect many (or any) business conversions from BEN to CC then we could do nothing ... but that's an @OlgaPotiagalova decision. What's the risk if this does happen?

How much effort is creating an equivalent filing type in COLIN?

@vysakh-menon-aot
Copy link
Collaborator

Impact: Will fail to sync the specific alteration and subsequent filings in that business.

Home team has to introduce this filing type and update all the relevant messaging (BN, MRAS, or any other). Once its in place we can update colin-api with the new filing type.

@vishnup0422
Copy link
Collaborator

vishnup0422 commented Nov 13, 2024

BC -> BEN

BC Business: BC0883711

image.png

BC-> CC

Private Zenhub Video

BEN -> BC

image.png

BC -> ULC

Screenshot 2024-11-13 at 12.07.22 PM.png

Private Zenhub Video

ULC -> BC

image.png

BEN -> CC

image.png

Private Zenhub Video

ULC -> BEN

image.png

image.png

@vishnup0422
Copy link
Collaborator

vishnup0422 commented Nov 13, 2024

Continued In Limited Company -> CCC

image.png

image.png

Private Zenhub Video

Continued In BEN -> CCC

image.png

Private Zenhub Video

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 - Lorrie A label to filter on the tickets for the Entities based team that David is PO for. Priority2
Projects
None yet
Development

No branches or pull requests

7 participants