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

[LockDown] Cog redoing #42

Open
SharkyTheKing opened this issue Sep 11, 2020 · 4 comments
Open

[LockDown] Cog redoing #42

SharkyTheKing opened this issue Sep 11, 2020 · 4 comments
Assignees
Labels
Priority: Low Status: Blocked Blocked due to concerns or whatever reason. Type: Enhancement

Comments

@SharkyTheKing
Copy link
Owner

Thanks to @kablekompany for making a PR -> #41

I'll be using this issue to list and detail changes that I'll be making towards Lockdown.

Current list:

  • Account for multiple channel addition/deletion.
  • Remove cross-guild lockdown (not needed, impossible to account for situations and not worth the effort. Discord stability has been reliable enough to not need this for servers)
  • Account for console logging
  • Account for channel notifications.
  • Account for both Text and Voice channels being able to be locked
  • Account for lockdown messages and unlockdown messages not being able to be sent, return message to author saying where they aren't able to be sent.
@SharkyTheKing
Copy link
Owner Author

Linking to PR. #41

I won't be merging this completely, though I will be taking aspects from this PR to expand on from this current code.

@SharkyTheKing
Copy link
Owner Author

Started working on this, to see the first commit review. cd47235

To keep an eye on it, keep an eye out for https://github.com/SharkyTheKing/Sharky/tree/recode/lockdown

@SharkyTheKing
Copy link
Owner Author

#44 was merged and is published, though I'll continue to work on this and add new features as I can.

@SharkyTheKing
Copy link
Owner Author

Probably going to add in a lockdown's ReadMe in the cog file so I can keep track of everything I need to do.

@SharkyTheKing SharkyTheKing added Priority: Low Status: Blocked Blocked due to concerns or whatever reason. Type: Enhancement and removed Feature Request labels Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: Low Status: Blocked Blocked due to concerns or whatever reason. Type: Enhancement
Projects
None yet
Development

No branches or pull requests

1 participant