Skip to content

[CHA-0] Fix QueryReminders to use filter parameter instead of filter_conditions#360

Merged
nijeesh-stream merged 1 commit intomasterfrom
fix_reminders_query
Nov 19, 2025
Merged

[CHA-0] Fix QueryReminders to use filter parameter instead of filter_conditions#360
nijeesh-stream merged 1 commit intomasterfrom
fix_reminders_query

Conversation

@nijeesh-stream
Copy link
Contributor

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

Changes

  • Fixed QueryReminders API call to use filter parameter instead of filter_conditions
  • Updated request body formatting for consistency

Why

The API expects the parameter name filter, not filter_conditions. This fix aligns the SDK with the actual API specification.

Testing

  • ✅ All existing tests pass
  • ✅ Manually verified filters work correctly with production staging credentials
  • ✅ Tested message_id, channel_cid, and combined filters

@nijeesh-stream nijeesh-stream enabled auto-merge (squash) November 19, 2025 17:46
@nijeesh-stream nijeesh-stream merged commit d00843e into master Nov 19, 2025
7 of 8 checks passed
@nijeesh-stream nijeesh-stream deleted the fix_reminders_query branch November 19, 2025 17:48
@github-actions github-actions bot mentioned this pull request Dec 19, 2025
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