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

Implement subscribe command #19

Open
2 tasks
SageStarCodes opened this issue May 14, 2018 · 3 comments
Open
2 tasks

Implement subscribe command #19

SageStarCodes opened this issue May 14, 2018 · 3 comments
Labels
enhancement New feature or request wontfix This will not be worked on

Comments

@SageStarCodes
Copy link
Contributor

SageStarCodes commented May 14, 2018

Implement a command that allows users to:

  • subscribe to new posts by the Node-Videos-Alert using a role
  • unsubscribe users from the posts by removing the role

OR

Add information to the #welcome channel describing how to enable notifications for the #new-videos channel.

@SageStarCodes SageStarCodes added the enhancement New feature or request label May 14, 2018
@SageStarCodes SageStarCodes added this to the 1.1.0 milestone May 14, 2018
@SageStarCodes SageStarCodes changed the title Implement subscribe command Implement subscribe command May 14, 2018
@SageStarCodes SageStarCodes added the dev This issue is related to the dev branch label May 14, 2018
@jemand771
Copy link
Contributor

we could also save the snowflake ids of subscribed users in the db instead of using a role. Though i dont see any advantages/disadvantages in doing so

@SageStarCodes
Copy link
Contributor Author

How would you notify the users even if you had their IDs?

@SageStarCodes
Copy link
Contributor Author

Hm... I just realized this is not an issue for E-D-G-E, we shouldn't rely on having a specific role and specific webhook available to us

@SageStarCodes SageStarCodes added the wontfix This will not be worked on label May 15, 2018
@SageStarCodes SageStarCodes removed this from the 1.1.0 milestone May 15, 2018
@SageStarCodes SageStarCodes removed the dev This issue is related to the dev branch label May 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants