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

Microsoft Mail: Add save_attachments to get_email_by_id #116

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

mikahanninen
Copy link
Contributor

@mikahanninen mikahanninen commented Jan 21, 2025

Description

Add the possibility of saving email attachments to a local folder.

How added parameter save_attachments works:

  • by default will save into the current working directory.
  • if the value downloads is given then attachments are saved in the user's Downloads folder
  • can be given local file path, on make_dirs=True this directory structure will be created if it does not exist

How can (was) this tested?

Tested in VS Code and in the Studio.

Screenshots (if needed)

Checklist:

  • I have bumped the version number for the Action Package / Agent
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation - README.md file
  • I have updated the CHANGELOG.md file in correspondence with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • OAuth2: I have made sure that action has necessary scopes (works in whitelisted mode)

@mikahanninen mikahanninen self-assigned this Jan 21, 2025
@mikahanninen mikahanninen added the enhancement New feature or request label Jan 21, 2025
@mikahanninen mikahanninen merged commit 0724bca into main Jan 21, 2025
6 checks passed
@mikahanninen mikahanninen deleted the microsoft-mail-add-save-attachments branch January 21, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants