Bug 1583821

Summary: Notification events are out of order
Product: Red Hat CloudForms Management Engine Reporter: Ryan Spagnola <rspagnol>
Component: UI - OPSAssignee: Dávid Halász <dhalasz>
Status: CLOSED CURRENTRELEASE QA Contact: Matouš Mojžíš <mmojzis>
Severity: medium Docs Contact:
Priority: high    
Version: 5.8.0CC: cpelland, hkataria, lavenel, mmojzis, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.10.0   
Hardware: All   
OS: All   
Whiteboard: ui:notification
Fixed In Version: 5.10.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1586213 1586214 (view as bug list) Environment:
Last Closed: 2019-02-11 14:08:38 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1586213, 1586214    

Description Ryan Spagnola 2018-05-29 19:10:14 UTC 
Description of problem:
Continuation of this bz: https://bugzilla.redhat.com/show_bug.cgi?id=1479922

Notifications appear in order during session. After closing and opening UI the notifications are presented out of order

Version-Release number of selected component (if applicable):
5.8.2.3

How reproducible:
Always

Steps to Reproduce:
1. Perform an action that presents a notification. ex: enable Embedded Ansible
2. After both notifications appear close browser tab and open new 
3. click notification bell

Actual results:
latest notification is pushed to the bottom of the list

Expected results:
latest notification should remain at the top

Additional info:
Comment 2 Ryan Spagnola 2018-05-31 12:28:49 UTC 
This can be easily reproduced by logging into an appliance that has pre-existing notifications. The sort order for older notifications will be oldest at the top.
Comment 3 CFME Bot 2018-05-31 12:38:08 UTC 
https://github.com/ManageIQ/manageiq-ui-classic/pull/4010
Comment 4 CFME Bot 2018-05-31 17:35:33 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/b7f7ad2e2d3daa9c78ae6694bf0d6f59e3202cd8
commit b7f7ad2e2d3daa9c78ae6694bf0d6f59e3202cd8
Author:     Dávid Halász <dhalasz>
AuthorDate: Thu May 31 08:16:59 2018 -0400
Commit:     Dávid Halász <dhalasz>
CommitDate: Thu May 31 08:16:59 2018 -0400

    Render notifications fetched from the API in the correct order

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

 app/assets/javascripts/services/event_notifications_service.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 7 Matouš Mojžíš 2018-06-19 16:43:44 UTC 
One time issue. Setting qe_test_coverage flag to -.
Comment 8 Matouš Mojžíš 2018-07-24 13:11:36 UTC 
Verified in 5.10.0.4. Notification events are in order even after logout/login(close tab/login).