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

[stable12] Fix notifications order #93

Merged
merged 2 commits into from Sep 27, 2017

Conversation

Projects
None yet
2 participants
@nickvergessen
Member

nickvergessen commented Sep 26, 2017

Backport #91

nickvergessen added some commits Aug 29, 2017

Append notifications on initial fetch
but prepend them when grabbing later

Signed-off-by: Joas Schilling <coding@schilljs.com>
Resort the list by timestamp after adding new items on the fly
Signed-off-by: Joas Schilling <coding@schilljs.com>
@MorrisJobke

Tested and works 👍

@MorrisJobke MorrisJobke merged commit 5dc2983 into stable12 Sep 27, 2017

3 checks passed

continuous-integration/drone/pr the build was successful
Details
continuous-integration/drone/push the build was successful
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@MorrisJobke MorrisJobke deleted the backport-91-fix-notifications-order branch Sep 27, 2017

@MorrisJobke MorrisJobke referenced this pull request Nov 20, 2017

Merged

12.0.4 RC #7225

1 of 1 task complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment