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

User Registration Validation Lists Email Missing Error Twice #16619

Closed
SusanEnneking opened this issue Aug 26, 2024 · 6 comments
Closed

User Registration Validation Lists Email Missing Error Twice #16619

SusanEnneking opened this issue Aug 26, 2024 · 6 comments
Labels

Comments

@SusanEnneking
Copy link

Describe the bug

Registration produces duplicate "Email is required" messages when the submit button is pressed without entering any information.

Orchard Core version

1.8.3

To Reproduce

Steps to reproduce the behavior:

  1. The problem can be reproduced using Try Orchard Core
  2. Choose the Agency Theme and enable Registration
  3. Navigate to /Registration
  4. Click submit without filling in any input and you'll see the error.

Looks like this problem has occurred in the past and was addressed with this ticket.

Expected behavior

When Email address is missing, the validation error should only show up once.

Logs and screenshots

Try Orchard Core 2024-08-24 at 8 12 30 AM
Copy link
Contributor

Thank you for submitting your first issue, awesome! 🚀 We're thrilled to receive your input. If you haven't completed the template yet, please take a moment to do so. This ensures that we fully understand your feature request or bug report. On what happens next, see the docs.

If you like Orchard Core, please star our repo and join our community channels.

@OrchardCMS OrchardCMS deleted a comment Aug 26, 2024
@OrchardCMS OrchardCMS deleted a comment Aug 26, 2024
@OrchardCMS OrchardCMS deleted a comment Aug 26, 2024
@OrchardCMS OrchardCMS deleted a comment from masooddahmedd Aug 26, 2024
@MikeAlhayek
Copy link
Member

This issue was fixed in 2.0 via #16552

@Piedone Piedone closed this as completed Aug 26, 2024
@gvkries
Copy link
Contributor

gvkries commented Aug 27, 2024

This issue was fixed in 2.0 via #16552

Nope, that was a different issue.

I remember that I fixed this issue a long time ago, but it was re-introduced shortly afterwards by #11130 which I did not notice.

But now in 2.0 the whole login/registration has been changed to use shapes. I think that change fixes this issue as well. Or can you reproduce this in main?

@gvkries gvkries reopened this Aug 27, 2024
@MikeAlhayek
Copy link
Member

@gvkries the issue is fixed in main. Is there a reason you re-opened this issue?

@gvkries
Copy link
Contributor

gvkries commented Aug 28, 2024

@gvkries the issue is fixed in main. Is there a reason you re-opened this issue?

Because I haven't had time to actually check if my statement above is correct...

@gvkries
Copy link
Contributor

gvkries commented Aug 28, 2024

This issue has been fixed in 2.0, but by #15736.

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

5 participants
@Piedone @SusanEnneking @gvkries @MikeAlhayek and others