bcgov / entity

ServiceBC Registry Team working on Legal Entities
Apache License 2.0
23 stars 58 forks source link

Edit UI: Cont. In company can only alter to another Cont. In type company (C, CBEN, CCC, CUL) #23092

Open severinbeauvais opened 2 weeks ago

severinbeauvais commented 2 weeks ago

See also #21669.

In #22479, a conversion was applied so that a C company can alter to a regular type (BC, BEN, CC, ULC). This was incorrect. THIS ticket is to change this behaviour. See this PR that made the original change.

Scenarios:

Here are the test scenarios that were performed when the company type change feature was implemented:

image.png
severinbeauvais commented 2 weeks ago

deleted obsolete comment

vikas-gov commented 2 weeks ago

deleted obsolete comment

OlgaPotiagalova commented 1 week ago

~Confirmed with the Business that C company should be converted to BC, BEN, CC, ULC. So the C prefix in the company type is not carried over in the alteration process~ see comments below

severinbeauvais commented 1 week ago

deleted obsolete comment

vysakh-menon-aot commented 1 week ago

@OlgaPotiagalova I assume there won't be any change in the identifier (C1234567)

Mihai-QuickSilverDev commented 1 week ago

@vysakh-menon-aot The identifier changes. As per Olga - "So the C prefix in the company type is not carried over in the alteration process". By extrapolation, we do not identify BC companies with a C number.

vysakh-menon-aot commented 1 week ago

@Mihai-QuickSilverDev can we verify that in colin once?

OlgaPotiagalova commented 1 week ago

I assume, the identifier shouldn't be affected. @Mihai-QuickSilverDev, could you please confirm this using COLIN data. Thank you.

Mihai-QuickSilverDev commented 1 week ago
severinbeauvais commented 1 week ago

There are still conflicting requirements. This will be discussed with the business on Tuesday, and then we can figure out what code needs to change.

per @Mihai-QuickSilverDev and @OlgaPotiagalova

Mihai-QuickSilverDev commented 1 week ago
vikas-gov commented 3 days ago

need further analysis and review before prioritizing

severinbeauvais commented 3 days ago

deleted obsolete comment

Mihai-QuickSilverDev commented 3 days ago
severinbeauvais commented 3 days ago

It looks like the "you need a Name Request to make this change" code is already in place, so theoretically this functionality should work correctly for continued in businesses (C, etc) as well as regular businesses (BC, etc). I am going to assume that this current functionality is satisfactory and no changes to it are needed.

Example:

image.png

https://dev.edit.business.bcregistry.gov.bc.ca/BC0883068/alteration?accountid=3040

ArwenQin commented 1 day ago

The issues previously in #23076 are solved in this ticket. The issue that unable to alter the company for valid legal types (Cont in ULC to benefit company) was for Cont in businesses only. Now fixed, Cont in can only be altered to a Cont in.