revert: exclude metadata when refetching feed after new message received#857
Conversation
|
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| "inserted_at.gte"?: string; | ||
| "inserted_at.lte"?: string; | ||
| "inserted_at.gt"?: string; | ||
| "inserted_at.lt"?: string; |
There was a problem hiding this comment.
Thomas asked the same question here. I added these in #853 because they were missing from the FetchFeedOptionsForRequest type. I can add them back in a separate PR, though I don’t think it’s much of an issue, since the FetchFeedOptionsForRequest type is only meant for internal use by the client SDK.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #857 +/- ##
==========================================
+ Coverage 68.35% 68.45% +0.09%
==========================================
Files 193 193
Lines 8122 8062 -60
Branches 1076 1065 -11
==========================================
- Hits 5552 5519 -33
+ Misses 2550 2518 -32
- Partials 20 25 +5
|
This reverts PR #853 (commit 1e75952). We decided against releasing this change; see this Slack thread for context.
Checklist