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

UI Bug - LETA - Staff able to move to Review and Confirm without deleting all owners and added ADMIN #18382

Closed
tlebedovich opened this issue Oct 30, 2023 · 3 comments
Assignees
Labels
Assets bug Something isn't working

Comments

@tlebedovich
Copy link

tlebedovich commented Oct 30, 2023

in DEV

MHR 107835

When I start a Transfer to Administrator (LETA)
And there are multiple owners
Staff needs to remove all existing owners and add an administrator
Validation message appears in homeowners table
but staff is not stopped from going to review and confirm screen when there are errors (they should be stopped)

To Fix:

  • fix validation on Review and Confirm button
  • missing button validation message, red bar, autoscroll and
  • errors should stop staff from going to review and confirm when there are errors on first screen

Private Zenhub Image

@tlebedovich tlebedovich added Assets bug Something isn't working labels Oct 30, 2023
@tlebedovich tlebedovich changed the title UI Bug - Staff allowed to move to Review and Confirm without deleting all owners and added ADMIN UI Bug - LETA - Staff able to move to Review and Confirm without deleting all owners and added ADMIN Oct 30, 2023
@dimak1 dimak1 self-assigned this Mar 20, 2024
@dimak1
Copy link
Collaborator

dimak1 commented Mar 21, 2024

@LizGovier @saragunnarsson in case you wanted to do a quick verification: https://bcregistry-assets-dev--pr-1794-kdt26ink.web.app

@saragunnarsson
Copy link
Collaborator

Thanks @dimak1 That looks great to me:)

@chdivyareddy
Copy link
Collaborator

Verified in DEV!

image.png

image.png

image.png

@PCC199 PCC199 closed this as completed Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Assets bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants