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

Add email action #82

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Add email action #82

wants to merge 2 commits into from

Conversation

tfeldmann
Copy link
Owner

Adds an action to send a file as an email.
One can use template variables in subject and body.
As the simulation output for this action does not fit into a single line the action base class is extended to support multiline output.

@tfeldmann
Copy link
Owner Author

tfeldmann commented Jun 12, 2020

Although this is working some decisions are still open:

  • Password should not be in the config file. Load from environment variables?
  • sim / run output?
  • Global settings dict with defaults in the config file? Otherwise you repeat parameters a lot.
  • Allow template variables in the other parameters?

Also missing:

  • Documentation
  • A good __str__

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant