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

Load initial chat messages from most recent to oldest. #1322

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@fancycode
Member

fancycode commented Nov 29, 2018

This implements #1150.

@danxuliu The sorting of entries is wrong: each chunk is sorted correctly in itself, but the overall list is sorted wrong (most recent are at the top). I'm not familiar with the chat message / virtual list code, so could you please take a look?

Load initial chat messages from most recent to oldest.
Signed-off-by: Joachim Bauch <bauch@struktur.de>
@danxuliu

This comment has been minimized.

Member

danxuliu commented Nov 29, 2018

@fancycode Sorry, I had this mostly done already :-S I will take a look anyway, but I will probably close this in favour of what I worked on (once I fix one pesky bug with the virtual list). Sorry again :-(

@nickvergessen nickvergessen added this to the 💔 Backlog milestone Dec 11, 2018

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