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

Filings UI: disable local corrections #21846

Closed
2 of 3 tasks
severinbeauvais opened this issue Jun 17, 2024 · 11 comments
Closed
2 of 3 tasks

Filings UI: disable local corrections #21846

severinbeauvais opened this issue Jun 17, 2024 · 11 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

Comments

@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Jun 17, 2024

New description

  • disable the "correct this filing" menu item for the 17 selected filings listed below (see comments)
  • should we delete the local correction code or leave it unused? left in place for now
  • update (or disable?) unit tests

Original description

An Annual Report local correction was successfully filed (but got stuck -- different issue):

image.png

An AGM Location Change local correction could not be filed:

image.png

A Consent Continuation Out local correction could be filed:

image.png

Here are the filings that are corrected "locally" (in Filings UI) instead of redirecting to the Edit UI correction page:

  • AGM Extension
  • AGM Location Change
  • Amalgamation Out
  • Annual Report
  • Change of Address
  • Change of Directors
  • Change of Company Info (ie, firms)
  • Change of Name
  • Consent Amalgamation Out
  • Consent Continuation Out
  • Continuation In
  • Continuation Out
  • Conversion (ie, firms)
  • Dissolution
  • Dissolved
  • Restoration
  • Transition

(Staff filings cannot be corrected.)

@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 Jun 17, 2024
@severinbeauvais severinbeauvais changed the title Filings UI: some local corrections don't work Business schema: some local corrections don't work Jun 17, 2024
@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Jun 17, 2024

Root Cause

The correction schema does not list some of the above filings as supported.

@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Jun 17, 2024

@NaveenHebbale @OlgaPotiagalova Can you please determine whether we should allow the correction of the missing filing types?

Filing type Supported in schema?
AGM Extension NO
AGM Location Change NO
Amalgamation Out NO
Annual Report y
Change of Address y
Change of Directors y
Change of Company Info NO
Change of Name y
Consent Amalgamation Out NO
Consent Continuation Out NO
Continuation In y
Continuation Out NO (but y to Continued Out)
Conversion y
Dissolution y
Dissolved y
Restoration y
Transition NO

@severinbeauvais
Copy link
Collaborator Author

Of the list in the table, above:

  • should they be "local corrections" or redirect to the Edit UI correction page?
  • should we simply block those corrections?

@severinbeauvais
Copy link
Collaborator Author

This ticket is blocked while awaiting the requirements.

@NaveenHebbale
Copy link
Collaborator

@severinbeauvais, Yesterday I discussed with @OlgaPotiagalova .For MVP we will disable correction for AR- Annual Report, AGM extension, AGM Location Change, etc .

@severinbeauvais severinbeauvais changed the title Business schema: some local corrections don't work Filings UI: disable local corrections Jun 19, 2024
@severinbeauvais severinbeauvais self-assigned this Jun 19, 2024
@severinbeauvais
Copy link
Collaborator Author

When this ticket is implemented, please ensure that tickets #21672 and #21799 are closed.

@severinbeauvais
Copy link
Collaborator Author

@NaveenHebbale Should we allow a correction to a Continuation In? After all, it's similar to an Incorporation Application, Registration, or Amalgamation.

@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Jun 19, 2024

@NaveenHebbale Please confirm that this is correct.

In this ticket, I will disable the correction of the following filings:

  • AGM Extension
  • AGM Location Change
  • Amalgamation Out
  • Annual Report
  • Change of Company Info
  • Consent Amalgamation out
  • Consent Continuation Out
  • Continuation Out
  • Conversion
  • Dissolution
  • Dissolved
  • Restoration
  • Transition

Corrections will be enabled for the following filings:

  • Alteration
  • Amalgamation Application
  • Change of Address
  • Change of Directors
  • Change of Name
  • Change of Registration
  • Continuation In
  • Correction
  • Incorporation Application
  • Registration
  • Special Resolution disabled for now

@NaveenHebbale
Copy link
Collaborator

@severinbeauvais
Disabled Option - I agree
Enabled Option - I need to double-check for a Special resolution

@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Jun 20, 2024

Test Notes

Try some or all of these. The unit tests cover all scenarios.

image.png

@NaveenHebbale
Copy link
Collaborator

Closed

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
Projects
None yet
Development

No branches or pull requests

2 participants