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

Validate Help->Documentation screen against the documentation feature #3882

Merged
merged 1 commit into from
May 3, 2018

Conversation

skateman
Copy link
Member

@skateman skateman commented May 2, 2018

The naming of methods/actions inside this controller (its name included) is just horrible and misleading. The RBAC check was improperly introduced in #3728 against the support feature, but it should be documentation.

@miq-bot add_reviewer @romanblanco
@miq-bot add_reviewer @martinpovolny
ping @simaishi

@miq-bot add_label bug, gaprindashvili/yes, fine/yes

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1563241
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1573996

@miq-bot
Copy link
Member

miq-bot commented May 2, 2018

Checked commit skateman@b1a79fa with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

@martinpovolny
Copy link
Member

documentation exists in db/fixtures/miq_product_features.yml, support does not.

Looks good this time ;-)

@martinpovolny martinpovolny self-assigned this May 3, 2018
@martinpovolny martinpovolny merged commit 2b4ec4c into ManageIQ:master May 3, 2018
@martinpovolny martinpovolny added this to the Sprint 85 Ending May 7, 2018 milestone May 3, 2018
@skateman skateman deleted the fix-rbac-about branch May 3, 2018 13:18
simaishi pushed a commit that referenced this pull request May 7, 2018
Validate Help->Documentation screen against the documentation feature
(cherry picked from commit 2b4ec4c)

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

simaishi commented May 7, 2018

Fine backport details:

$ git log -1
commit d86d1b8e604a0b341ead02295439eb0a6811d5b5
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Thu May 3 15:14:29 2018 +0200

    Merge pull request #3882 from skateman/fix-rbac-about
    
    Validate Help->Documentation screen against the documentation feature
    (cherry picked from commit 2b4ec4c82fa3cb4d191ec156a87acc0229ee3034)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1569171
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1575714

simaishi pushed a commit that referenced this pull request May 14, 2018
Validate Help->Documentation screen against the documentation feature
(cherry picked from commit 2b4ec4c)

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

Gaprindashvili backport details:

$ git log -1
commit cd57a6ab3b753877b3a6f43287ca8b238a90fa47
Author: Martin Povolny <mpovolny@redhat.com>
Date:   Thu May 3 15:14:29 2018 +0200

    Merge pull request #3882 from skateman/fix-rbac-about
    
    Validate Help->Documentation screen against the documentation feature
    (cherry picked from commit 2b4ec4c82fa3cb4d191ec156a87acc0229ee3034)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1569170
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1575713

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