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 registration type field highlighted in admin meeting creation form #9160
Fix registration type field highlighted in admin meeting creation form #9160
Conversation
… message before form completion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm that this is working as expected locally.
I'm not sure if we should also add a regression test for this one, probably at
it "creates a new meeting", :slow do |
Oh and thanks for the PR @roxanaopr!! This was the typical small thing that whenever I found it I said to myself "I need to check that" but didn't have the chance, so great that you've done it! 👏🏽 👏🏽 |
39ab604
to
5692d02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽 LGTM, thanks for the PR!
#9160) * Remove trigger from meeting_form.js in order to avoid rendering error message before form completion * Test that the error is no longer displayed on form.
🎩 What? Why?
Registration type field is already highlighted upon creating a meeting via admin
📌 Related Issues
Testing