- Notifications
You must be signed in to change notification settings - Fork1.1k
fix: renametotal tocount inAIBridgeListInterceptionsResponse#20410
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| typeAIBridgeListInterceptionsResponsestruct { | ||
| Totalint64`json:"total"` | ||
| Countint64`json:"count"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
It'll need to be changed in test files too. I suggest undoing your changes and then doing an automated rename refactor operation in vscode (which should change it everywhere). Then re-runmake gen
d455f6e intomainUh oh!
There was an error while loading.Please reload this page.
Thanks to the great work in#20393, we’ve successfully introduced offset-based pagination for this endpoint. However, the frontend expects a
countfield in the response rather thantotal. This PR updates the response payload to rename the returned key tocountfor consistency with frontend expectations and existing API patterns.This is necessary to unblock the work in#20331