Skip to content

feat: add reminders #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
Closed
wants to merge 22 commits into from
Closed

feat: add reminders #1496

wants to merge 22 commits into from

Conversation

nijeesh-stream
Copy link
Contributor

@nijeesh-stream nijeesh-stream commented Mar 13, 2025 •
edited by MartinCupela
Loading

Goal

For the reminders the following is added:

  • Reminder types
  • API request methods
  • reactive state managed by RemindersManager
  • timers for reminders
  • reminder pagination with reactive state

@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: +10.4 kB (+2.52%)

Total Size: 425 kB

Filename Size Change
dist/cjs/index.browser.cjs 122 kB +3.47 kB (+2.91%)
dist/cjs/index.node.cjs 167 kB +3.53 kB (+2.17%)
dist/esm/index.js 135 kB +3.43 kB (+2.6%)

compressed-size-action

MartinCupela
MartinCupela reviewed May 9, 2025
View reviewed changes
@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
@MartinCupela MartinCupela removed the Backend Team The backend team is in charge of this issue label Jun 2, 2025
@MartinCupela MartinCupela changed the title feat: implemented the reminders feature feat: add reminders Jun 3, 2025
@MartinCupela MartinCupela mentioned this pull request Jun 3, 2025
Merged
isekovanic
isekovanic reviewed Jun 4, 2025
View reviewed changes
@MartinCupela MartinCupela requested a review from isekovanic June 4, 2025 12:38
isekovanic
isekovanic approved these changes Jun 4, 2025
View reviewed changes
Copy link
Contributor

@isekovanic isekovanic left a comment

Choose a reason for hiding this comment

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

Looks great to me ! Thanks !

@MartinCupela MartinCupela deleted the feature/snooze_message_reminder branch June 4, 2025 14:27
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

@isekovanic isekovanic isekovanic approved these changes

@arnautov-anton arnautov-anton arnautov-anton 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

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

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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