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

bz 1509244: Don't require form changes in order to submit backup restore #3911

Merged
merged 1 commit into from
May 14, 2018

Conversation

sseago
Copy link
Contributor

@sseago sseago commented May 8, 2018

https://bugzilla.redhat.com/show_bug.cgi?id=1509244

The 'save' button in the backup restore form wasn't active until
the user first selected a different volume. To restore to the
current volume required changing to a different volume and then
re-selecting the original. This commit removes the requirement
to change the form first before submitting. Such a requirement
makes sense for an edit form (no changes, no need to submit),
but it does not make sense for an action form like this.

@sseago sseago closed this May 9, 2018
@sseago sseago reopened this May 9, 2018
@sseago sseago closed this May 10, 2018
@sseago sseago reopened this May 10, 2018
https://bugzilla.redhat.com/show_bug.cgi?id=1509244

The 'save' button in the backup restore form wasn't active until
the user first selected a different volume. To restore to the
current volume required changing to a different volume and then
re-selecting the original. This commit removes the requirement
to change the form first before submitting. Such a requirement
makes sense for an edit form (no changes, no need to submit),
but it does not make sense for an action form like this.
@miq-bot
Copy link
Member

miq-bot commented May 10, 2018

Checked commit sseago@34bddd8 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
0 files checked, 0 offenses detected
Everything looks fine. 👍

@sseago sseago closed this May 11, 2018
@sseago sseago reopened this May 11, 2018
@mzazrivec mzazrivec self-assigned this May 14, 2018
@mzazrivec mzazrivec added this to the Sprint 86 Ending May 21, 2018 milestone May 14, 2018
@mzazrivec mzazrivec merged commit 2e5eb63 into ManageIQ:master May 14, 2018
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.

3 participants