Skip to content

Fixes #612 embed images in emails#731

Merged
oharsta merged 7 commits into
mainfrom
feature/612-embed-images-in-emails
May 22, 2026
Merged

Fixes #612 embed images in emails#731
oharsta merged 7 commits into
mainfrom
feature/612-embed-images-in-emails

Conversation

@ricardovdheijden
Copy link
Copy Markdown
Contributor

@ricardovdheijden ricardovdheijden commented May 21, 2026

This PR only handles the external content, that is the institution logo. The other 2 logos in the email are added to the email as attachment

  • Includes a hardcoded limit of 1MB for the logo
  • Handles malformed images: no logo will be added

@ricardovdheijden ricardovdheijden marked this pull request as draft May 21, 2026 09:22
@phavekes phavekes moved this from New to In Progress in Openconext-Invite May 21, 2026
@ricardovdheijden ricardovdheijden marked this pull request as ready for review May 22, 2026 11:01
@ricardovdheijden ricardovdheijden force-pushed the feature/612-embed-images-in-emails branch from 7ec29f2 to 685d00e Compare May 22, 2026 11:02
@oharsta oharsta merged commit 10fcbdd into main May 22, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Delivered in Openconext-Invite May 22, 2026
@oharsta oharsta deleted the feature/612-embed-images-in-emails branch May 22, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Delivered

Development

Successfully merging this pull request may close these issues.

3 participants