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

Fix screen updates in RH Updates screen #2774

Merged

Conversation

mzazrivec
Copy link
Contributor

  1. Configuration -> Settings -> Region -> Red Hat Updates
  2. 'Refresh list' button should refresh the screen
  3. All the other buttons should enqueue a task + render a flash message

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

@mzazrivec mzazrivec force-pushed the fix_screen_updates_in_rh_updates_screen branch from 0103358 to f1deb8e Compare November 20, 2017 16:12
@miq-bot
Copy link
Member

miq-bot commented Nov 20, 2017

Checked commit mzazrivec@f1deb8e with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@dclarizio dclarizio merged commit 17aaf4b into ManageIQ:master Nov 21, 2017
@dclarizio dclarizio added this to the Sprint 74 Ending Nov 27, 2017 milestone Nov 21, 2017
@mzazrivec mzazrivec deleted the fix_screen_updates_in_rh_updates_screen branch November 22, 2017 05:47
simaishi pushed a commit that referenced this pull request Nov 27, 2017
…tes_screen

Fix screen updates in RH Updates screen
(cherry picked from commit 17aaf4b)

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

Gaprindashvili backport details:

$ git log -1
commit 9a514d27765a0f560e2f434bf954097c54964166
Author: Dan Clarizio <dclarizi@redhat.com>
Date:   Tue Nov 21 14:28:06 2017 -0800

    Merge pull request #2774 from mzazrivec/fix_screen_updates_in_rh_updates_screen
    
    Fix screen updates in RH Updates screen
    (cherry picked from commit 17aaf4bc9fe5722bd4e4b32445e7bc5cc91fe339)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1517880

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.

5 participants