surface-security / django-notification-sender

Django App to let users configure a notification system in Surface. Supports Email and Slack
MIT License
1 stars 1 forks source link

rename NOTIFICATIONS_SLACK_APP_TOKEN to NOTIFICATIONS_SLACK_BOT_TOKEN #7

Open fopinappb opened 2 years ago

fopinappb commented 2 years ago

using APP_TOKEN is confusing as the Slack app configuration has two distinct tokens: one called bot token (that is what we need) and another called app token used for socket mode (which is not what we need).

Asking for the bot token in a variable wit APP_TOKEN is confusing :)

Support both settings temporarily, raise deprecation warning?