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

Account Requests - Fix Users disappearing on error #4523

Closed
cielf opened this issue Jul 14, 2024 · 4 comments · Fixed by #4779
Closed

Account Requests - Fix Users disappearing on error #4523

cielf opened this issue Jul 14, 2024 · 4 comments · Fixed by #4779

Comments

@cielf
Copy link
Collaborator

cielf commented Jul 14, 2024

Summary

If a super admin is editing an account request and attempts to save it, but there is an error, the user information disappears. This shouldn't happen.

Why fix?

Possible error in re-entering, also slows down the process

Criteria for completion

  • [] User information is preserved if there is an error on the account request edit
  • [] tests to support this behaviour.
@awwaiid awwaiid modified the milestone: Tasks 2024 Jul 28, 2024
@coalest
Copy link
Collaborator

coalest commented Nov 12, 2024

@cielf Can you provide some more information on how to replicate this issue? I don't see an option for admins to edit account requests.

@cielf
Copy link
Collaborator Author

cielf commented Nov 12, 2024

@coalest
On local, from the home page. Click "Register your bank here for free". Click "I am an Essentials Banks..." Fill in a name, email, details and "I'm not a robot", and submit. Wait a couple of minutes for the email to appear in your browser.
Click "Confirm this Request", then "I'm Ready, Let's go!" Wait a couple of minutes for the next email to appear in your browser. Click "Create This Organization". Sign in with the superadmin login

That should take you to a screen where you are editing that particular account request. If you scroll down, you will see a user. Click save without filling in a short name. There will be an error (expected) and the user will be gone (not expected).

@coalest
Copy link
Collaborator

coalest commented Nov 13, 2024

@cielf Thanks for the explanation! Can I take this task?

@cielf
Copy link
Collaborator Author

cielf commented Nov 13, 2024

Yup!

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

Successfully merging a pull request may close this issue.

3 participants