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

Fix error handling on registration #3426

Closed
2 tasks
cielf opened this issue Mar 5, 2023 · 15 comments · Fixed by #3883
Closed
2 tasks

Fix error handling on registration #3426

cielf opened this issue Mar 5, 2023 · 15 comments · Fixed by #3883
Assignees
Labels

Comments

@cielf
Copy link
Collaborator

cielf commented Mar 5, 2023

Summary

Mistakes on the registration form die without feedback. Fix this.

Why fix it?

Making onboarding less error-prone. Reducing support load

Details

Recreating the problem

On local, , Click on "Register your bank here for free", Choose "I am an Essentials Bank" . Fill in all the mandatory fields except request details. Submit.

It goes to the screen with the choice between bank and partner, With no feedback that anything is wrong. (It actually flashes something, but too quick to read.)

It should, instead, go to the screen with the registration details. There should be an error message at the top, as well as the errors that currently show there if you re-choose bank.

Updated knowledge

Apparently it does quickly flash the expected screen, but then goes to the screen with the choice between bank and partner

Criteria for completion

  • If there is an error on filling out the registration form, the registration details appear, with error messaging.
  • tests to support this behaviour
@cielf cielf added Help Wanted Groomed + open to all! 🐞 Bug labels Mar 5, 2023
@cielf
Copy link
Collaborator Author

cielf commented Mar 8, 2023

This looks like a javascript thing on the new.html.

@rosenwu
Copy link

rosenwu commented Mar 29, 2023

@cielf Hi, I think we can try this, can you assign this to me?

@dorner
Copy link
Collaborator

dorner commented Mar 29, 2023

You got it!

@github-actions
Copy link
Contributor

github-actions bot commented Apr 9, 2023

This issue has been inactive for 243 hours (10.13 days) and will be automatically unassigned after 117 more hours (4.88 days).

@github-actions
Copy link
Contributor

This issue has been inactive for 363 hours (15.13 days) and is past the limit of 360 hours (15.00 days) so is being unassigned.

@nadineah
Copy link

Can I take a shot at this issue?

@cielf
Copy link
Collaborator Author

cielf commented Apr 20, 2023

@nadineah Go for it!

@lisawan2-cmu-F24
Copy link

Hi! I think also there is a similar issue when submitting a blank registration form. It will go back to the original state. And then after clicking on the "I am an Essentials Bank" radio button again, the error appears.

@cielf
Copy link
Collaborator Author

cielf commented Apr 24, 2023

@Gemini010815 Indeed -- I think that a blank registration form would be a mistake, and would hopefully be covered by the fix for this.

@github-actions
Copy link
Contributor

github-actions bot commented May 5, 2023

This issue has been inactive for 245 hours (10.21 days) and will be automatically unassigned after 115 more hours (4.79 days).

@github-actions
Copy link
Contributor

This issue has been inactive for 365 hours (15.21 days) and is past the limit of 360 hours (15.00 days) so is being unassigned.

@westonio
Copy link
Contributor

Hi @cielf & @scooter-dangle , I recently joined RFG in Slack and would love to take this on as my first issue.

I experimented on my end and already have a working fix for this. Would you mind assigning this one to me?

@dorner
Copy link
Collaborator

dorner commented Sep 14, 2023

Go for it!

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label Sep 14, 2023
@westonio
Copy link
Contributor

westonio commented Sep 14, 2023

Awesome, thank you for the help getting this assigned to me @dorner 😄 🙌

@github-actions
Copy link
Contributor

This issue is marked as stale due to no activity within 30 days. If no further activity is detected within 7 days, it will be unassigned.

@github-actions github-actions bot added stale and removed stale labels Oct 15, 2023
dorner pushed a commit that referenced this issue Oct 27, 2023
* Fix account request error handling & view

* Fix: add flash to account_requests_controller and update javascript actions on account_requests new.html

* Refactor: Remove erroneous flash message and update script to select Essentials Bank to reveal the form with errors

* Fix: remove unnecessary flash message I had added for account_requests #create action

* Fix: Add parameter & controller method to reload form on render :new

* Refactor: Remove erroneous methods, variables, and script for re-rendering form with errors

* Fix: Add closing div tag
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants