-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
UX: Messages: Use "Newest" as default mode #579
Conversation
resolves #570 |
i tried to solve the problem this way. as a result "Newest" is displayed in the UI, but sometimes the messages are empty. only changing to "Oldest" and then back to "Newest" helps. |
should be working, as we have |
Based on my testing, everything is functioning properly, and I don't see any reason for this issue to occur. |
I tested the change again with a variety of messages and was unable to reproduce the issue. Additionally, I reviewed the codebase for any other references to the "Newest" filter and found no related components. Given this, I believe there isn't a valid reason to withhold approval for this pull request. If the issue continues to affect you, I recommend opening a new issue with more details to help us investigate further :) |
@polegkashti thank you! |
What changes did you make? (Give an overview)
Simply chaned the default filter mode.
Is there anything you'd like reviewers to focus on?
Hi, I’ve noticed that other users have also expressed a need for this change, and some of our Kafka users specifically mentioned it. I believe it’s a great idea to modify the default mode, as it aligns better with Kafka's real-time streaming nature.
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)
Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)
Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)