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

Feature : Added the feature of sending the notification of latest video/note to discord #502

Closed

Conversation

amanbairagi30
Copy link
Contributor

@amanbairagi30 amanbairagi30 commented Apr 28, 2024

Problem Statement

Currently, whenever a new video is uploaded into the CMS, Harkirat has to update the Discord community about it. This process requires additional effort after posting the video in the CMS.

Solution

I have implemented a solution where whenever a new video or note is added from the CMS admin panel, a notification for that video or note can be seen in Discord. I have made this an optional choice. If moderators want to add a video/note without sending a notification to discord, they can do so. Additionally , I have provided a checkbox for this purpose . Along with that in discord message you can even go to that particular video /note location by clicking the link provided there .

You can see the changes below :

2024-04-28.16-11-19.mp4

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

@TanmayDhobale
Copy link
Contributor

damn @amanbairagi30

@amanbairagi30
Copy link
Contributor Author

@hkirat Please review this PR .

@ichiragkumar
Copy link

ichiragkumar commented May 9, 2024

great job, it was on my list but you did this add sending notification in Gmail or WhatsApp, any one which is cheaper

@amanbairagi30
Copy link
Contributor Author

great job, it was on my list but you did this add sending notification in Gmail or WhatsApp, any one which is cheaper

Initially, I too wanted to do that with Gmail but after researching a bit I got to know that it comes with a lot of cost so I decided to go with discord .

@amanbairagi30
Copy link
Contributor Author

Closing because of some weird merge conflicts and raised a new PR at #601

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.

4 participants