-
Notifications
You must be signed in to change notification settings - Fork 13
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
Add flag to toggle_profile_activation
command to turn all off
#756
Comments
I think you can combine this one and #757 and toggle_notification_profiles into a single command. "list" is incompatible with the others, it might even be the fallback if there are no flags. |
I have trouble picturing how that would look like? That seems un-intuitive for me, because what if we want to later add another management command that does something else with notification profiles and there it might also be useful to know all profiles, then we would have to double the code or call this command before even though we don't want to do anything with the activation. I would prefer to have these separate and then we can mention the management command in the helptext for |
Add flag where? To what? What are we talking about here? |
toggle_profile_activation
command to turn all off
This can then also be added to the documentation introduced in #765 |
Originally posted by @hmpf in #749 (review)
The text was updated successfully, but these errors were encountered: