Skip to content

Add broadcasts.get_by_user endpoint to get broadcasts created by a user#149

Merged
kraktus merged 1 commit intolichess-org:masterfrom
DoraFgr:feature/broadcasts-get-by-user
Dec 2, 2025
Merged

Add broadcasts.get_by_user endpoint to get broadcasts created by a user#149
kraktus merged 1 commit intolichess-org:masterfrom
DoraFgr:feature/broadcasts-get-by-user

Conversation

@DoraFgr
Copy link
Copy Markdown
Contributor

@DoraFgr DoraFgr commented Dec 1, 2025

This PR is a small part of #6 and implements broadcasts.get_by_user() endpoint to retrieve broadcasts created by a specific user

Summary:

  • Implement client.broadcasts.get_by_user() to fetch paginated broadcasts by username
  • Add BroadcastsByUser TypedDict for response typing
  • Add test with VCR cassette validation
Checklist when adding a new endpoint
  • Added new endpoint to the README.md
  • Ensured that my endpoint name does not repeat the name of the client. Wrong: client.users.get_user(), Correct: client.users.get()
  • Typed the returned JSON using TypedDicts in berserk/types/, example
  • Written tests for GET endpoints not requiring authentification. Documentation, example
  • Added the endpoint and your name to CHANGELOG.md in the To be released section (to be created if necessary)

@DoraFgr
Copy link
Copy Markdown
Contributor Author

DoraFgr commented Dec 1, 2025

There seem to be an issue with the CI test. I opened #150 to futher discuss the issue.

@DoraFgr DoraFgr marked this pull request as ready for review December 1, 2025 02:53
@DoraFgr
Copy link
Copy Markdown
Contributor Author

DoraFgr commented Dec 2, 2025

I don’t seem to have the required permission to rerun the CI. If someone can rerun the jobs when reviewing, that would be great.

@kraktus kraktus merged commit 1818488 into lichess-org:master Dec 2, 2025
31 of 36 checks passed
@kraktus
Copy link
Copy Markdown
Member

kraktus commented Dec 2, 2025

thanks! clean PR 👍

@kraktus kraktus mentioned this pull request Dec 3, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants