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 Mailgun Agent #2225
base: nightly
Are you sure you want to change the base?
Add Mailgun Agent #2225
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also write up the Mailgun instructions to add to the wiki?
https://github.com/Tautulli/Tautulli/wiki/Notification-Agents-Guide
self.NAME, "Missing recipient email address") | ||
return False | ||
|
||
# Get emails addresses from Mailgun API |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Factor out getting address into a separate method. agent_notify
should only handle sending the email.
{'label': 'BCC Recipients', | ||
'value': self.config['bcc_recipients'], | ||
'name': 'mailgun_bcc_recipients', | ||
'description': 'Add users to Bcc instead of To.', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'description': 'Add users to Bcc instead of To.', | |
'description': 'Add users to BCC instead of To.', |
{'label': 'API Key', | ||
'value': self.config['api_key'], | ||
'name': 'mailgun_api_key', | ||
'description': 'API key for Mailgun.', | ||
'input_type': 'password' | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move API key up so it is the first setting.
{'label': 'API Key', | |
'value': self.config['api_key'], | |
'name': 'mailgun_api_key', | |
'description': 'API key for Mailgun.', | |
'input_type': 'password' | |
}, | |
{'label': 'Mailgun API Key', | |
'value': self.config['api_key'], | |
'name': 'mailgun_api_key', | |
'description': 'Your Mailgun API key.', | |
'input_type': 'token' | |
}, |
Description
Adds a notification agent for Mailgun using the Mailgun API and allows using it for newsletters
Adds the ability to:
Screenshot
Type of Change
Checklist