bcgov / entity

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

Create UI: resolution or court order dates should be carried across in an amalgamation #22846

Closed severinbeauvais closed 5 days ago

severinbeauvais commented 2 months ago

In #20893, two BENs were amalgamated into a new Benefit Company. The bug is that a resolution date (pertaining to the share structure) from the primary TING was not carried over to the new TED.

See also #23054 (back end ticket).

severinbeauvais commented 1 month ago

deleted obsolete comment

severinbeauvais commented 1 month ago

To do:

severinbeauvais commented 2 weeks ago

The PR for this work has been merged. UX peeps, please have a quick look and pass along to QA if satisfactory. (I think Ketaki already checked with you on styling. You can also check the PR for screenshots.)

jacqueline-williams-549 commented 2 weeks ago

I did review with Ketaki and the styling for this looks great. I had some concerns about the language, but was unable to get it reviewed/approved today.

jacqueline-williams-549 commented 1 week ago

I would like to simplify the language used here

Original statement

Would like to change to this

@ketaki-deodhar This language has been approved

janisrogers commented 1 week ago

I don't know that your statement conveys the same information. Let's discuss.

ketaki-deodhar commented 1 week ago

@jacqueline-williams-549 I will update it soon

jacqueline-williams-549 commented 1 week ago

Thanks @ketaki-deodhar this looks great.