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

Add Notifications Service and some notifications #3825

Merged
merged 8 commits into from
Jun 5, 2023

Conversation

javfg
Copy link
Contributor

@javfg javfg commented Apr 27, 2023

This PR adds the notification service to Reva. It is to be reviewed after the Serverless Services one.

Due to the size, it is divided in multiple commits which add separate stuff:

  • Notifications service.
  • Add notification helper to some ownCloud services used by web.
  • Add notifications when uploading files to a uploader public link.
  • Add notifications when creating a share.
  • Remove old mailer service.

This requires changes in the DB, which are detailed in this file.

@javfg javfg requested review from a team, labkode, gmgigi96 and glpatcern as code owners April 27, 2023 13:55
@update-docs
Copy link

update-docs bot commented Apr 27, 2023

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@javfg javfg force-pushed the notifications-cs3org branch 7 times, most recently from 726e6f2 to 498c9e3 Compare May 12, 2023 13:43
@labkode labkode merged commit 61723ff into cs3org:master Jun 5, 2023
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Jun 5, 2023
gmgigi96 pushed a commit to gmgigi96/reva that referenced this pull request Jun 28, 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