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

Sign-up Form Validation message correction #3340

Closed
3 tasks
grvsachdeva opened this issue Sep 16, 2018 · 14 comments
Closed
3 tasks

Sign-up Form Validation message correction #3340

grvsachdeva opened this issue Sep 16, 2018 · 14 comments

Comments

@grvsachdeva
Copy link
Member

grvsachdeva commented Sep 16, 2018

We are preparing to participate in Google Code-in, and have reserved this issue for participants in GCI - but we'd love to have your help with another one! Please check out https://code.publiclab.org to see more.

Problem

When a user tries to submit an empty sign up form, and also does not check off the recaptcha box, This output is shown:

image

Solution

Instead, we want all the relevant errors ("field cannot be empty" or so) to be listed along with the failed recaptcha verification.

Each box in this problem is worth some points. We will accept answers that complete one part and not the other, but encourage you to go for all 3. Completion of all three will result in bonus points :)

  • fix the create action so that the correct errors show
  • fix the error messages themselves as well. If you see below, there are three different errors for email, but only one of them is correct.
  • write a test reflecting this fix. Guidelines: the test must fail without your changes, and your changes are what makes it pass.

image

@grvsachdeva
Copy link
Member Author

Hi, this issue has been opened as a task on GCI dashboard. If you are Google Code-In participant and want to solve this issue, please claim it from Google Code-In website . Thanks!

@tsparksh tsparksh mentioned this issue Oct 25, 2018
@venkateshdb

This comment has been minimized.

@jywarren

This comment has been minimized.

@SidharthBansal

This comment has been minimized.

@Dsxv

This comment has been minimized.

@SidharthBansal

This comment has been minimized.

@SidharthBansal SidharthBansal added rgsoc Rails Girls Summer of Code gsoc labels Jan 11, 2019
@ananyaarun

This comment has been minimized.

@grvsachdeva

This comment has been minimized.

@SidharthBansal

This comment has been minimized.

@sashadev-sky sashadev-sky removed gsoc rgsoc Rails Girls Summer of Code labels Sep 13, 2019
@publiclab publiclab deleted a comment from raddhika Sep 13, 2019
@publiclab publiclab deleted a comment from Paarmita Sep 13, 2019
@publiclab publiclab deleted a comment from oorjitchowdhary Sep 13, 2019
@publiclab publiclab deleted a comment from SidharthBansal Sep 13, 2019
@publiclab publiclab deleted a comment from SidharthBansal Sep 13, 2019
@publiclab publiclab deleted a comment from avsingh999 Sep 13, 2019
@publiclab publiclab deleted a comment from venkateshdb Sep 13, 2019
@publiclab publiclab deleted a comment from venkateshdb Sep 13, 2019
@publiclab publiclab deleted a comment from SidharthBansal Sep 13, 2019
@publiclab publiclab deleted a comment from grvsachdeva Sep 13, 2019
@publiclab publiclab deleted a comment from oorjitchowdhary Sep 13, 2019
@publiclab publiclab deleted a comment from oorjitchowdhary Sep 13, 2019
@publiclab publiclab deleted a comment from SidharthBansal Sep 13, 2019
@publiclab publiclab deleted a comment from SidharthBansal Sep 13, 2019
@publiclab publiclab deleted a comment from SidharthBansal Sep 13, 2019
@sashadev-sky
Copy link
Member

@SidharthBansal @gauravano I thought I was cleaning the issue by deleting comments but it makes it look so cluttered still!! Noted

@sashadev-sky
Copy link
Member

@SidharthBansal @gauravano we need to add code links here. Since you opened the original issue @gauravano would you be able to figure out where the changes need to be made? The old link you provided is not a valid link anymore!

@SidharthBansal
Copy link
Member

SidharthBansal commented Sep 13, 2019 via email

@sashadev-sky
Copy link
Member

Ok if we want to keep it can either you or @gauravano please re-open with code links :)

@SidharthBansal
Copy link
Member

Moved to #6303
@sashadev-sky it will be great if you use hide feature instead of delete feature.
Thanks for help!

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