Skip to content

Add events API - #276

Merged
jessegeens merged 1 commit into
mainfrom
feat/notifications
Jul 23, 2026
Merged

Add events API#276
jessegeens merged 1 commit into
mainfrom
feat/notifications

Conversation

@jessegeens

@jessegeens jessegeens commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

Hello,

We would like to propose an event-based notifications API in the CS3APIs.

We are reworking our implementation of notifications in Reva and we think this is a nice way to handle it. The idea is that there are certain "events" in reva that a user might want to be notified of (e.g. a share is created, file is uploaded in a folder, etc). There are many ways these can be handled (email, push notification, etc.). Users might also have preferences for how often they want to be contacted, and for certain event types you also want to accumulate some events (e.g. a user uploading 100 files).

We decided to make it more generic, so that events can also be used to trigger other actions. With this API, it is completely up to the handler to handle all of this; all you set is an event_type and some data and the implementation then chooses what to do with it (we will use a system where you can define rules for event types in the config).

What do you think (@aduffeck @micbar @butonic)?

@jessegeens
jessegeens force-pushed the feat/notifications branch from 28a73bb to 008fcea Compare July 14, 2026 09:05
@jessegeens
jessegeens marked this pull request as ready for review July 16, 2026 08:19
@jessegeens
jessegeens marked this pull request as draft July 16, 2026 14:29
@jessegeens
jessegeens force-pushed the feat/notifications branch 3 times, most recently from 02f271a to 8d4f8a5 Compare July 20, 2026 13:54
@jessegeens jessegeens changed the title Add notifications API Add events API Jul 20, 2026
@jessegeens
jessegeens force-pushed the feat/notifications branch from 8d4f8a5 to fc465f1 Compare July 20, 2026 15:01
@jessegeens
jessegeens marked this pull request as ready for review July 20, 2026 15:01

@glpatcern glpatcern left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we can go ahead with this non-breaking change (let alone the need to define stubs if not implemented).

@jessegeens
jessegeens merged commit c05101a into main Jul 23, 2026
2 checks passed
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