Skip to content
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: Add type field support on create_blocklist() #149

Merged
merged 4 commits into from
Nov 23, 2023

Conversation

viktorapo808
Copy link
Contributor

Submit a pull request

CLA

  • I have signed the Stream CLA (required).
  • The code changes follow best practices
  • Code changes are tested (add some information if not applicable)

Description of the pull request

For the regex blocklists we need to introduce a new type field in the DB to be able to distinguish between regular and regex blocklist (type). Therefore, when creating blocklist through the dashboard/SDK we need to pass the type field in the payload. In future, ideally this should support more types.
With the current change I extended the create_blocklist() method with an optional parameter that should pass the type field to the request payload

@viktorapo808 viktorapo808 requested a review from gumuz as a code owner November 14, 2023 11:48
stream_chat/async_chat/client.py Outdated Show resolved Hide resolved
stream_chat/client.py Outdated Show resolved Hide resolved
stream_chat/tests/test_client.py Outdated Show resolved Hide resolved
@viktorapo808 viktorapo808 force-pushed the add-type-support-on-create-blocklist branch from 90ad390 to 1167cd8 Compare November 16, 2023 14:49
@viktorapo808 viktorapo808 merged commit 786a50c into master Nov 23, 2023
5 checks passed
@viktorapo808 viktorapo808 deleted the add-type-support-on-create-blocklist branch November 23, 2023 12:21
@github-actions github-actions bot mentioned this pull request Nov 23, 2023
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