Skip to content

feat: implemented the reminders feature #1496

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

feat: implemented the reminders feature #1496

wants to merge 10 commits into from

Conversation

nijeesh-stream
Copy link

CLA

  • I have signed the Stream CLA (required).
  • Code changes are tested

Description of the changes, What, Why and How?

Implemented the reminders feature

  • reminders create
  • reminders delete
  • reminders update
  • query reminders

@nijeesh-stream nijeesh-stream marked this pull request as draft March 13, 2025 14:22
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 13, 2025 •
edited
Loading

Size Change: +706 B (+0.17%)

Total Size: 415 kB

Filename Size Change
dist/cjs/index.browser.cjs 119 kB +233 B (+0.2%)
dist/cjs/index.node.cjs 163 kB +240 B (+0.15%)
dist/esm/index.js 132 kB +233 B (+0.18%)

compressed-size-action

MartinCupela
MartinCupela reviewed May 9, 2025
View reviewed changes
src/client.ts Outdated
* @param {UpdateReminderOptions} options The options for updating the reminder
* @returns {Promise<ReminderResponse>}
*/
async updateReminder<StreamChatGenerics extends ExtendableGenerics = DefaultGenerics>(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nijeesh-stream please remove the generics as they are not supported by the SDK anymore. Thank you

@arnautov-anton arnautov-anton added the Backend Team The backend team is in charge of this issue label May 27, 2025
@totalimmersion totalimmersion marked this pull request as ready for review May 29, 2025 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MartinCupela MartinCupela MartinCupela left review comments

@totalimmersion totalimmersion totalimmersion approved these changes

@vishalnarkhede vishalnarkhede Awaiting requested review from vishalnarkhede vishalnarkhede is a code owner

@santhoshvai santhoshvai Awaiting requested review from santhoshvai santhoshvai is a code owner

@khushal87 khushal87 Awaiting requested review from khushal87 khushal87 is a code owner

@szuperaz szuperaz Awaiting requested review from szuperaz szuperaz is a code owner

@oliverlaz oliverlaz Awaiting requested review from oliverlaz oliverlaz is a code owner

@arnautov-anton arnautov-anton Awaiting requested review from arnautov-anton arnautov-anton is a code owner

@myandrienko myandrienko Awaiting requested review from myandrienko myandrienko is a code owner

Assignees
No one assigned
Labels
Backend Team The backend team is in charge of this issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nijeesh-stream @totalimmersion @MartinCupela @arnautov-anton @hifaizsk