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
41 stars 36 forks source link

[GSSoC - 23] feat: Separating the functionality of Notification button to add & delete roles. #42

Closed JatinSharma32 closed 1 year ago

JatinSharma32 commented 1 year ago

This PR is the implementation of Separating the functionality of Notification button to add & delete roles. issue

Related Issue

Closes #34

How Has This Been Tested?

Screenshots (if appropriate):

Provided in #34

Description:

Sorry, it just went out of my head to disable the Prettier which exploring the Codebase. I intended to only format ./interactions/notifications.js & ./commands/setup-contest.js but ended up formatting 5-6 files for no reason. It's a silly mistake I will take care of it next time.

JatinSharma32 commented 1 year ago

Yup surely 🙂

JatinSharma32 commented 1 year ago

Fixed those formatting issues in my latest commit (ca9ba8b)

JatinSharma32 commented 1 year ago

Are you considering this Pull Request or i have to make a new one, because I have already made the requested changes so please have a look.