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

Button Group: Fix 'Display on Button' checkbox visibility #3431

Merged
merged 1 commit into from
Feb 20, 2018

Conversation

mzazrivec
Copy link
Contributor

This is addressing problem described in #3383 (comment) and #3424 (comment)

  1. Add / Edit of a Button Group should not produce error
  2. Display on Button checkbox should be consistent in both the edit & details screen.

[true, "1"] because historically you'll find both in the database.

@himdel

@miq-bot
Copy link
Member

miq-bot commented Feb 20, 2018

Checked commit mzazrivec@55e7cd3 with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Haml - Linter::Haml STDERR:
warning: parser/current is loading parser/ruby23, which recognizes
warning: 2.3.5-compliant syntax, but you are running 2.3.3.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.

@himdel
Copy link
Contributor

himdel commented Feb 20, 2018

LGTM, verified that this

  • doesn't crash
  • the checkbox is consistent across edits (_group_form) and when viewing (_ab_list)

👍

(kicking travis and waiting :))

@himdel himdel merged commit d3819e4 into ManageIQ:master Feb 20, 2018
@himdel himdel added this to the Sprint 80 Ending Feb 26, 2018 milestone Feb 20, 2018
@mzazrivec mzazrivec deleted the fix_display_on_button_checkbox branch February 22, 2018 12:10
simaishi pushed a commit that referenced this pull request Mar 8, 2018
Button Group: Fix 'Display on Button' checkbox visibility
(cherry picked from commit d3819e4)

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

simaishi commented Mar 8, 2018

Gaprindashvili backport details:

$ git log -1
commit 1d340c0bb3091222dfb878be51dd81004686f1cb
Author: Martin Hradil <[email protected]>
Date:   Tue Feb 20 16:04:36 2018 +0000

    Merge pull request #3431 from mzazrivec/fix_display_on_button_checkbox
    
    Button Group: Fix 'Display on Button' checkbox visibility
    (cherry picked from commit d3819e4d9fae5f823a59972452ae26b5267aae17)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1553214

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