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

Create UI: Amalgamating Businesses component - Part 3b (remaining validations) #18971

Closed
3 tasks done
severinbeauvais opened this issue Dec 12, 2023 · 8 comments
Closed
3 tasks done
Assignees
Labels

Comments

@severinbeauvais
Copy link
Collaborator

severinbeauvais commented Dec 12, 2023

This is the second part of ticket #18640. That ticket's estimate of 8 has been reduced to 5, and this ticket is estimated as 3.

  • implement basic validations (Yui's original table of 6 rules)
  • implement advanced amalgamating businesses validations (see Google doc link below)
  • add tooltip to "Ready" values (see business list in uxpin)

Google doc: https://docs.google.com/document/d/1Dt2jzPA2SLtQO7Ns2aPCzVchGdv4YrYB/edit

@severinbeauvais severinbeauvais self-assigned this Dec 12, 2023
@severinbeauvais severinbeauvais changed the title Create UI: implement new Amalgamating Businesses component - Part 3b (more validations) Create UI: Amalgamating Businesses component - Part 3b (remaining validations) Dec 12, 2023
@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Dec 13, 2023

@yuisotozaki This is the new ticket I'm working on. As part of this ticket:

  • what do you think about the snackbar?
  • what do you think about the spinner?

@yuisotozaki
Copy link

I have to pass the snackbar by the design team. I know we use them but accessibility support isn't good. We don't want to introduce widgets that we'll need to undo later.
As for the spinner - do we use it elsewhere?

@severinbeauvais
Copy link
Collaborator Author

severinbeauvais commented Dec 13, 2023

I'm open to alternate designs for add error messaging.

Yes, we use the spinner in multiple UI/filings when we need to fetch additional data from the API. << a good example of this is clicking "View Documents" for any ledger items -- it goes and fetches the documents list and any comments the first time you click on this item

@severinbeauvais
Copy link
Collaborator Author

@riyazuddinsyed This ticket is tagged to multiple releases because it has content in several releases (due to multiple, incremental PRs).

@severinbeauvais
Copy link
Collaborator Author

This is ready for UX Assurance.

Sample amalgamation applications (try as BCREG0020 and staff):
https://dev.create.business.bcregistry.gov.bc.ca/amalg-reg-information?id=Tc2caJYQrz
https://dev.create.business.bcregistry.gov.bc.ca/amalg-reg-information?id=TD2mgyjRd1
(or create your own!)

Yui, if you don't like the spinner or the snackbar, please create a new ticket. Thx.

For a detailed look at the implemented validation rules, see this code:
https://github.com/bcgov/business-create-ui/blob/main/src/mixins/amalgamation-mixin.ts#L25

@yuisotozaki
Copy link

Observation:

  • Replace the "Close" label on the snackbar with an ex icon in white. This is how the MBR list works when a new row is added successfully.

image.png

@severinbeauvais
Copy link
Collaborator Author

This will be done in #18788.

@yuisotozaki
Copy link

Ready for QA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants