-
Notifications
You must be signed in to change notification settings - Fork 390
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
feat: change default orderBy to message
#1519
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
size-limit report 📦
|
75a2c2e
to
2310019
Compare
2310019
to
96bc271
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## next #1519 +/- ##
==========================================
+ Coverage 74.11% 74.19% +0.08%
==========================================
Files 67 67
Lines 1862 1868 +6
Branches 492 494 +2
==========================================
+ Hits 1380 1386 +6
Misses 380 380
Partials 102 102
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
@taozhou-glean thank you! Could you please also add a section about this change to the v4 migration article? |
@andrii-bodnar added right below the |
Description
more context in #1515 (comment), in short, we want to switch default orderBy to the newly added
message
(source message) to match (more) with the behavior in v3.Types of changes
orderBy
Fixes # (issue) n/a
Checklist