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

New event hooks #50

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Conversation

padaliyajay
Copy link

No description provided.

@prlanzarin
Copy link
Member

Hi. Thanks for the contribution.
Can you please state the intended use cases for these events?
We also might need you to sign a CLA before accepting contributions - will have to check with higher ups.

@padaliyajay
Copy link
Author

Hi. Thanks for reply

Here is possible use case of these two events

1). event: presentation-annotated-pdf-ready

  • when participants export annotated presentation then it will send link in chat. but this annotated pdf is temporary store in server and remove after some days. in this case, we can use this event webhook to permanently store annotated presentation pdf in third party database so users can access it anytime in future.
  • another use case is, we can send email to each participant with download link when presentation exported

2). event: presentation-uploaded

  • this event trigger when moderator upload presentation. in this case we can keep track of used presentation pdf in third party database

@prlanzarin prlanzarin self-requested a review August 22, 2024 13:18
@prlanzarin
Copy link
Member

prlanzarin commented Sep 5, 2024

@padaliyajay It seems no CLA is needed for this.
I'll review + merge in the coming days, thanks for the contribution.

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