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

Fix for the infinte spinner on Playbook Catalog Item Edit Form Reset #3552

Merged
merged 1 commit into from Mar 12, 2018

Conversation

lgalis
Copy link
Contributor

@lgalis lgalis commented Mar 8, 2018

Add a check for the data and stop the spinner when the Reset button is pressed on the Ansible Playbook Catalog Item edit form.

Links

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

Steps for Testing/QA

  1. Edit a Playbook Catalog Item
  2. Make a change and press Reset
  • infinite spinner before this change

@lgalis
Copy link
Contributor Author

lgalis commented Mar 8, 2018

@miq-bot add_label bug

@lgalis
Copy link
Contributor Author

lgalis commented Mar 8, 2018

@h-kataria - please review

@miq-bot miq-bot added the bug label Mar 8, 2018
@lgalis lgalis force-pushed the reset_infinite_spinner_fix branch from 17b885e to e112af7 Compare March 8, 2018 23:38
@miq-bot
Copy link
Member

miq-bot commented Mar 8, 2018

Checked commit lgalis@e112af7 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. 👍

@lgalis lgalis closed this Mar 9, 2018
@lgalis lgalis reopened this Mar 9, 2018
@mzazrivec
Copy link
Contributor

@lgalis Is this gaprindashvili/yes ?

@lgalis
Copy link
Contributor Author

lgalis commented Mar 9, 2018

@mzazrivec - yes, I think the problem exists in gaprindashvili as well.
I'll create a BZ for it

@miq-bot add_label gaprindashvili/yes

@h-kataria h-kataria self-assigned this Mar 12, 2018
@h-kataria h-kataria added this to the Sprint 81 Ending Mar 12, 2018 milestone Mar 12, 2018
@h-kataria h-kataria merged commit e72ad4a into ManageIQ:master Mar 12, 2018
@lgalis lgalis deleted the reset_infinite_spinner_fix branch March 12, 2018 18:16
simaishi pushed a commit that referenced this pull request Mar 13, 2018
Fix for the infinte spinner on Playbook Catalog Item Edit Form Reset
(cherry picked from commit e72ad4a)

https://bugzilla.redhat.com/show_bug.cgi?id=1554823
@simaishi
Copy link
Contributor

Gaprindashvili backport details:

$ git log -1
commit 7b4d081327ade1590171e254ea91ba7bae28e1a0
Author: Harpreet Kataria <hkataria@redhat.com>
Date:   Mon Mar 12 13:46:26 2018 -0400

    Merge pull request #3552 from lgalis/reset_infinite_spinner_fix
    
    Fix for the infinte spinner on Playbook Catalog Item Edit Form Reset
    (cherry picked from commit e72ad4a7f7a3e70cb873dd9cbbb0871e41727d6b)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1554823

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

5 participants