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

Enforce that name and ems_id be present to submit AuthKeyPair creation form #8357

Merged

Conversation

mansam
Copy link
Contributor

@mansam mansam commented Apr 29, 2016

Don't enable the Add button unless at least a name and ems_id
have been entered in the form. The key field remains optional.

This was not being enforced before, allowing invalid form
submissions.

cc @durandom https://bugzilla.redhat.com/show_bug.cgi?id=1329427

Don't enable the Add button unless at least a name and ems_id
have been entered in the form. The key field remains optional.

This was not being enforced before, allowing invalid form
submissions.
@miq-bot
Copy link
Member

miq-bot commented Apr 29, 2016

Checked commit mansam@85d8d92 with ruby 2.2.3, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 0 offenses detected
Everything looks good. 🍰

@chessbyte
Copy link
Member

@dclarizio @h-kataria @martinpovolny please review

@martinpovolny martinpovolny merged commit 98e0379 into ManageIQ:master May 2, 2016
@martinpovolny martinpovolny added this to the Sprint 40 Ending May 9, 2016 milestone May 2, 2016
@chessbyte
Copy link
Member

@dclarizio @martinpovolny darga/yes or darga/no?

chessbyte pushed a commit that referenced this pull request May 2, 2016
…elds

Enforce that `name` and `ems_id` be present to submit AuthKeyPair creation form
(cherry picked from commit 98e0379)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants