-
Notifications
You must be signed in to change notification settings - Fork 4
docs(api-reference): update GET /api/chats response shape #227
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
Merged
Merged
Changes from all commits
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
ac5cd06
docs(api-reference): update GET /api/chats to session-scoped shape
arpitgupta1214 31b5ea3
docs(api-reference): keep artist_account_id non-deprecated on GET /ap…
arpitgupta1214 b4a3dcb
docs(api-reference): modify ChatRoom schema in place for session-scop…
arpitgupta1214 f77477c
docs(api-reference): drop internal vocab from GET /api/chats descript…
arpitgupta1214 c7b8c24
Merge branch 'main' into docs/get-chats-session-scoped
sweetmantech e710dd4
docs(chats): add artistId to ChatRoom response schema
sweetmantech 23a32eb
docs(chats): describe artist_account_id as a real filter
sweetmantech File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
P1: The
artist_account_idparameter description was changed to describe active filtering behavior, but the PR description states the parameter is currently inert. If the parameter is still inert, this description will mislead API consumers into believing the filter works when it has no effect. Either restore thecurrently has no effectcaveat (or add a similaraccepted but currently has no effectnote) to match the stated intent, or update the PR description to reflect that filtering is now active.Prompt for AI agents