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#1650693 If tagging is readonly we should not show the edit dropdown #1497

Merged

Conversation

AllenBW
Copy link
Member

@AllenBW AllenBW commented Nov 16, 2018

But if tagging is not read only, behavior is nominal

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

see, no more edit dropdown here (where we shouldn't have one)

screen shot 2018-11-16 at 4 59 04 pm

but we can still do tagging where we need to !

screen shot 2018-11-16 at 4 59 16 pm

But if tagging is not read only, behavior is nominal
@miq-bot
Copy link
Member

miq-bot commented Nov 16, 2018

Checked commit AllenBW@2300d49 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. ⭐

@himdel himdel added this to the Sprint 99 Ending Nov 19, 2018 milestone Nov 19, 2018
Copy link
Contributor

@himdel himdel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, tested in the UI :)

@himdel himdel merged commit 953635b into ManageIQ:master Nov 19, 2018
@AllenBW AllenBW deleted the BZ/#1650693-fix-tagging-readonly-issue branch November 19, 2018 15:37
simaishi pushed a commit that referenced this pull request Nov 19, 2018
…y-issue

BZ#1650693 If tagging is readonly we should not show the edit dropdown

(cherry picked from commit 953635b)

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

Hammer backport details:

$ git log -1
commit af29bdaaf19c57998565b1bf6a4f69fc13fe6ee5
Author: Martin Hradil <[email protected]>
Date:   Mon Nov 19 16:31:43 2018 +0100

    Merge pull request #1497 from AllenBW/BZ/#1650693-fix-tagging-readonly-issue
    
    BZ#1650693 If tagging is readonly we should not show the edit dropdown
    
    (cherry picked from commit 953635b5b8a21387f775f94af6b6c325200992de)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1650693

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