cms icon indicating copy to clipboard operation
cms copied to clipboard

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

Open amanbairagi30 opened this issue 1 year ago • 6 comments

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 :

https://github.com/code100x/cms/assets/118182376/e8655f37-9d6e-434f-8502-e4f42ce65e8a

Checklist before requesting a review

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

amanbairagi30 avatar May 13 '24 06:05 amanbairagi30

Hey @hkirat , I have made the necessary changes you asked in live stream , and creating a new PR as the old one has some weird merge conflicts . I would request you to review this PR .

amanbairagi30 avatar May 13 '24 06:05 amanbairagi30

@amanbairagi30

can you add instruction and update .env.example on what do we need to setup the discord webhook url?

Also solve the conflicts

siinghd avatar Jun 14 '24 03:06 siinghd

@amanbairagi30

can you add instruction and update .env.example on what do we need to setup the discord webhook url?

Also solve the conflicts

Done ⚡

amanbairagi30 avatar Jun 14 '24 04:06 amanbairagi30

@siinghd , you may check now .

amanbairagi30 avatar Jun 17 '24 17:06 amanbairagi30

@siinghd sir ji , please look into this .

amanbairagi30 avatar Jul 08 '24 18:07 amanbairagi30

@siinghd ?

amanbairagi30 avatar Aug 08 '24 17:08 amanbairagi30

closing it as it has been merged to main via another PR

amanbairagi30 avatar Sep 15 '24 09:09 amanbairagi30