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 the custom button set set_data hash links to the original buttons on copy #19169

Merged

Conversation

d-m-u
Copy link
Contributor

@d-m-u d-m-u commented Aug 19, 2019

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1740556
Also needed for https://bugzilla.redhat.com/show_bug.cgi?id=1737449

The button set copy needs to reset and then save the new button order.

@miq-bot add_label bug, ivanchuk/yes
@miq-bot assign @bdunne

@d-m-u d-m-u force-pushed the fixing_custom_button_set_copy_children branch from 48f4ac5 to 6d1b45a Compare August 19, 2019 19:47
@miq-bot
Copy link
Member

miq-bot commented Aug 19, 2019

Checked commit d-m-u@6d1b45a with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. 🍰

@bdunne bdunne merged commit 68f281d into ManageIQ:master Aug 20, 2019
@bdunne bdunne modified the milestones: Sprint 118 Ending Aug 19, 2019, Sprint 119 Ending Sep 2, 2019 Aug 20, 2019
@d-m-u
Copy link
Contributor Author

d-m-u commented Aug 21, 2019

I'd like to backport to h along with #18368 for https://bugzilla.redhat.com/show_bug.cgi?id=1737449

@d-m-u
Copy link
Contributor Author

d-m-u commented Aug 21, 2019

@miq-bot add_label hammer/yes

simaishi pushed a commit that referenced this pull request Aug 21, 2019
…ildren

Fix the custom button set set_data hash links to the original buttons on copy

(cherry picked from commit 68f281d)

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

Ivanchuk backport details:

$ git log -1
commit c4b2d8aa0425fa12649c7b9a1d772e7da596a98e
Author: Brandon Dunne <bdunne@redhat.com>
Date:   Tue Aug 20 13:15:40 2019 -0400

    Merge pull request #19169 from d-m-u/fixing_custom_button_set_copy_children
    
    Fix the custom button set set_data hash links to the original buttons on copy
    
    (cherry picked from commit 68f281d47fe19f84f0aacff9e8d764d69e315416)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1740556
    https://bugzilla.redhat.com/show_bug.cgi?id=1737449

simaishi pushed a commit that referenced this pull request Sep 5, 2019
…ildren

Fix the custom button set set_data hash links to the original buttons on copy

(cherry picked from commit 68f281d)

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

simaishi commented Sep 5, 2019

Hammer backport details:

$ git log -1
commit 103f8808b91b7276587f473f031fb42f8f202e89
Author: Brandon Dunne <bdunne@redhat.com>
Date:   Tue Aug 20 13:15:40 2019 -0400

    Merge pull request #19169 from d-m-u/fixing_custom_button_set_copy_children
    
    Fix the custom button set set_data hash links to the original buttons on copy
    
    (cherry picked from commit 68f281d47fe19f84f0aacff9e8d764d69e315416)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1745198

@d-m-u d-m-u deleted the fixing_custom_button_set_copy_children branch September 26, 2019 10:49
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.

4 participants