Skip to content

Conversation

@Liemine
Copy link
Contributor

@Liemine Liemine commented Jan 15, 2026

No description provided.

variables.put("useEduID", invitation.isEduIDOnly() && invitation.getIntendedAuthority().equals(Authority.GUEST));

Map<String, String> images = new HashMap<>();
if(invitation.isEduIDOnly() && invitation.getIntendedAuthority().equals(Authority.GUEST)) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The invitation does not have to be for a GUEST. All authorities are allowed. Remove the second and clause (and possible swap if / else and negation.

// }
// });
// }
// *** NIEUW: Voeg inline images toe via CID ***
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

English please


private void addInlineImages(MimeMessageHelper helper, Map<String, String> imagePathsMap) {
imagePathsMap.forEach((cid, resourcePath) -> {
try {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ik zou @SneakyThrows gebruiken en geen leeg catch block

@oharsta oharsta merged commit e34a20b into main Jan 15, 2026
4 checks passed
@oharsta oharsta deleted the feature/#549-improve-invite-emails-gmail-fix branch January 15, 2026 12:35
@github-project-automation github-project-automation bot moved this from New to Delivered in Openconext-Invite Jan 15, 2026
@phavekes phavekes moved this from Delivered to Ready for release in Openconext-Invite Jan 15, 2026
@phavekes phavekes moved this from Ready for release to External Testing in Openconext-Invite Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: External Testing

Development

Successfully merging this pull request may close these issues.

3 participants