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

Do not remember tab when downloading report from list view. #5964

Conversation

h-kataria
Copy link
Contributor

@miq-bot
Copy link
Member

miq-bot commented Aug 7, 2019

Checked commit h-kataria@0849d4a with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 🍪

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

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

The Seal of Approval

@mzazrivec mzazrivec added this to the Sprint 118 Ending Aug 19, 2019 milestone Aug 8, 2019
@mzazrivec mzazrivec merged commit f0ebf99 into ManageIQ:master Aug 8, 2019
simaishi pushed a commit that referenced this pull request Aug 8, 2019
…ading_from_list_view

Do not remember tab when downloading report from list view.

(cherry picked from commit f0ebf99)

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

simaishi commented Aug 8, 2019

Ivanchuk backport details:

$ git log -1
commit d87822e5956f78d8c29f0218f3f1d44a0efac972
Author: Milan Zázrivec <[email protected]>
Date:   Thu Aug 8 13:39:46 2019 +0200

    Merge pull request #5964 from h-kataria/dont_remember_tab_when_downloading_from_list_view
    
    Do not remember tab when downloading report from list view.
    
    (cherry picked from commit f0ebf9938fa86f58e728ab03e6039acaf0216a66)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1738664

@h-kataria h-kataria deleted the dont_remember_tab_when_downloading_from_list_view branch August 22, 2019 17:47
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