-
Notifications
You must be signed in to change notification settings - Fork 59
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
Middle Name of Directors not showing up in Directors correction for BEN's Alterations. #18238
Comments
This error is already in Production. Tested it on BC1444534. We did not receive complaints yet, however, it should be fixed. Priority between a 2 and a 3. |
Middle Name and Middle Initial label mismatch is likely at play here. |
@Mihai-QuickSilverDev will review for an epic to include this one into. |
@Mihai-QuickSilverDev - Epic for this one please. |
Done. |
Test NotesIt turns out that this error occurred in all filings, not just corp (BEN) corrections or alterations as reported above. This error was already fixed for (firm) change and (firm) conversion filings. Instead of replicating the fix in 9 different filings, I moved the fix to the LegalServices class (which is where the directors/parties are fetched). Therefore, this fix affects all the following filings. (I tested most of them locally already; see PR for screenshots).
|
Sample Businesses in Devhttps://dev.business.bcregistry.gov.bc.ca/BC0871417 (for alteration and corp correction) https://dev.business.bcregistry.gov.bc.ca/FM1052869 (for change and conversion) https://dev.business.bcregistry.gov.bc.ca/CP1002588 (for special resolution and coop correction) https://dev.business.bcregistry.gov.bc.ca/FM1053038 (for firm correction) |
The fix is merged into Dev and I have verified it there. Ready for QA! |
Describe the bug in the current situation
When staff members are making corrections to a Director Change filing, they may notice that if the director has a middle name, it doesn't appear during the correction process.
Steps to Reproduce
Steps to reproduce the behavior:
The text was updated successfully, but these errors were encountered: