search

dashroshan / coding-contests-companion

💪 Discord bot for coding contests info, notifications, and daily problems from codechef, leetcode, hackerrank, codeforces, atcoder, hackerearth, and geeksforgeeks
MIT License
43 stars 40 forks source link

[GSSoC - 23] Feat: Personalized Notifications. #50

Open JatinSharma32 opened 1 year ago

JatinSharma32 commented 1 year ago

Issue

Hey, @roshan1337d I am back with my notification obsession 😂, I would like to add an amazing UX-enhancing feature to this coding-contests-companion bot. Where the user could choose which platforms he wants to get notified for.

Current Behavior

Right now the user could either get notified for all of the platform contests or none of them.

Possible Solution

We could provide the user options in the Notification Embed to choose the platform, but reacting to the Embed.

Possible Implementation

Will add more, soon

:bulb: Request: Please assign this issue to me, additionally please help out with the yml issue templates added recently, I am new to them, or I can continue with Markdowns.

KamalSritha-04 commented 1 year ago

Can you please assign me this issue under GSSOC'23?

JatinSharma32 commented 1 year ago

Actually, I've already applied for the issue, as it's based on First Come first serve basis. But still we can wait for PA to choose one of us🙂

KamalSritha-04 commented 1 year ago

@JatinSharma32 ya okay.

dashroshan commented 1 year ago

@JatinSharma32 assigned it to you!

But I would suggest some changes to your idea:

  1. Replace get notified and stop notifications buttons with opt-in to all, and opt-out of all. Make those buttons blue and red, and in a separate row. For all other platforms make those buttons as toggle buttons (opt-in with 1st click, opt-out with 2nd click). Add those buttons in new rows of same message, don't use different message. You can add upto 25 buttons to a single message.

  2. A different role for each platform would make it difficult to setup for server admins. So you will need to improve setup-contests command to auto-create the roles instead of asking which ones to use. The stop command should also be modified to delete those roles at end.

Good luck 🚀

JatinSharma32 commented 1 year ago

sure

JatinSharma32 commented 1 year ago

I am sorry, I am not getting enough time to work on it, please reassign this issue to some one else

dashroshan commented 1 year ago

@JatinSharma32 Alright, no problem 👍 The issue is open for reassignment.

Saimurugeshwari commented 1 year ago

Kindly assign this to me , I love to work on this

tripat1803 commented 6 months ago

Hi @dashroshan I am looking forward to work on this new feature introduced by JatinSharma32.

kart2004 commented 6 months ago

Hi @dashroshan, can you please assign this issue to me under GSSoC'24?