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

Use the uib-tab parameter for switching tabs on the tasks screen #5054

Merged
merged 1 commit into from
Dec 7, 2018

Conversation

skateman
Copy link
Member

@skateman skateman commented Dec 6, 2018

This is a regression caused by the angular-bootstrap prefixing, the new parameter includes the uib- prefix, but the controller was looking for the tab parameter.

Before:
screenshot from 2018-12-06 16-39-02

After:
screenshot from 2018-12-06 16-38-04

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

@miq-bot add_reviewer @epwinchell
@miq-bot add_reviewer @himdel
@miq-bot add_label bug, hammer/yes

@miq-bot
Copy link
Member

miq-bot commented Dec 6, 2018

Checked commit skateman@ed98945 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. 🍪

Copy link
Contributor

@epwinchell epwinchell left a comment

Choose a reason for hiding this comment

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

Works fine.

@skateman
Copy link
Member Author

skateman commented Dec 7, 2018

@miq-bot assign @himdel

@mzazrivec mzazrivec added the tasks label Dec 7, 2018
@mzazrivec mzazrivec assigned mzazrivec and unassigned himdel Dec 7, 2018
@mzazrivec mzazrivec added this to the Sprint 101 Ending Dec 17, 2018 milestone Dec 7, 2018
@mzazrivec mzazrivec merged commit 9e541dc into ManageIQ:master Dec 7, 2018
@skateman skateman deleted the uib-tab-tasks branch December 7, 2018 09:05
simaishi pushed a commit that referenced this pull request Dec 10, 2018
Use the uib-tab parameter for switching tabs on the tasks screen

(cherry picked from commit 9e541dc)

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

Hammer backport details:

$ git log -1
commit e563b6d9dbfca9a231dd2c07673563a7992c6792
Author: Milan Zázrivec <mzazrivec@redhat.com>
Date:   Fri Dec 7 09:01:26 2018 +0100

    Merge pull request #5054 from skateman/uib-tab-tasks
    
    Use the uib-tab parameter for switching tabs on the tasks screen
    
    (cherry picked from commit 9e541dc5636e4d333396305357bc6dc00323cc0d)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1655749

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.

6 participants