Opened on 10/04/2017 at 03:14:10 PM
Closed on 10/11/2017 at 09:23:02 AM
#5827 closed defect (fixed)
notification.json is empty if lastVersion contains a variant group
Reported by: | wspee | Assignee: | |
---|---|---|---|
Priority: | Unknown | Milestone: | |
Module: | Sitescripts | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | no | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description (last modified by wspee)
Environment
Filter server with:
- Sitecripts: f85775239239
- Infrastructure: 95ca18258f78
Chrome 61 ABP 1.13.4
How to reproduce
- Create and push the following notification to the filter server:
severity = information [1] title.en-US = Test message.en-US = Test sample = 1
- Start the browser and let ABP download the notification.
- Disable the previously created notification and add a new notification with the content from above.
- Let ABP download the notification again
Observed behaviour
The server delivers the following notification.json:
{ "notifications": [], "version": "201710041503-4/1" }
Expected behaviour
The server should deliver the following notification.json:
{ "notifications": [ { "id": "5", "message": { "en-US": "Test" }, "severity": "information", "title": { "en-US": "Test" } } ], "version": "201710041504-4/1-5/1" }
Note
To not break the AB machinery the server is supposed to deliver the same notification he delivered last time. But only if the same notification is still active and it shouldn't affect other notifications.
Attachments (0)
Change History (4)
comment:2 Changed on 10/09/2017 at 09:12:28 AM by wspee
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:3 Changed on 10/11/2017 at 09:20:43 AM by abpbot
A commit referencing this issue has landed:
Issue 5827 - Assign new groups even if prior groups are present
comment:4 Changed on 10/11/2017 at 09:23:02 AM by wspee
- Resolution set to fixed
- Status changed from reviewing to closed
Note: See
TracTickets for help on using
tickets.
The old group should be remembered.