From 6fc3fdbb43abf40b35bf103ce9dacc7efb60b5fb Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Sun, 4 Jan 2026 00:27:11 +0100 Subject: [PATCH 1/8] #549 Improve invite email --- .../resources/templates/eduID-logo-square.svg | 8 ++++++ .../resources/templates/invitation_en.html | 23 ++++++---------- .../resources/templates/invitation_nl.html | 26 +++++++------------ 3 files changed, 25 insertions(+), 32 deletions(-) create mode 100644 server/src/main/resources/templates/eduID-logo-square.svg diff --git a/server/src/main/resources/templates/eduID-logo-square.svg b/server/src/main/resources/templates/eduID-logo-square.svg new file mode 100644 index 00000000..c5930564 --- /dev/null +++ b/server/src/main/resources/templates/eduID-logo-square.svg @@ -0,0 +1,8 @@ + + + + + + + + diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index 13f8d389..6abf6e27 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -24,12 +24,7 @@ {{#invitation.anyRoles}} -

{{displaySenderName}} (from {{institutionName}}) has invited you for one or - more - applications or systems that they use.

- -

You have been invited for:

- +

{{displaySenderName}} (from {{institutionName}}) would you like to grant access to:

{{/invitation.anyRoles}} {{#message}} -

Personal message from {{displaySenderName}}

-
+

{{{message}}}

{{/message}} {{#useEduID}} -

For access to these applications eduID is used. Do you not have an eduID account yet? - No worries, you can create one in less than 30 seconds.

+

You accept the role(s) by logging in with eduID. If you don’t have one yet, you can create it right away.

- {{> eduid-logo.svg}} - Accept invitation with eduID + style="border-radius: 4px;color: white;display: flex;width: 200px;background-color: #0077c8;text-decoration: none;cursor:pointer;padding: 8px 20px 8px 10px;"> + {{> eduID-logo-square.svg}} + Continue with eduID
{{/useEduID}} @@ -60,8 +53,8 @@

For access to these applications we use SURFconext.

- Accept invitation + style="border-radius: 4px;color: white;display: flex;width: 200px;background-color: #0077c8;text-decoration: none;cursor:pointer;padding: 8px 20px 8px 10px;"> + Continue
{{/useEduID}} diff --git a/server/src/main/resources/templates/invitation_nl.html b/server/src/main/resources/templates/invitation_nl.html index c02cd223..c3fed347 100644 --- a/server/src/main/resources/templates/invitation_nl.html +++ b/server/src/main/resources/templates/invitation_nl.html @@ -24,12 +24,7 @@ {{#invitation.anyRoles}} -

{{displaySenderName}} (van {{institutionName}}) heeft je uitgenodigd voor - één of meer - applicaties en systemen die zij gebruiken.

- -

Je bent uitgenodigd voor:

- +

{{displaySenderName}} (van {{institutionName}}) wil je toegang geven tot:

{{/invitation.anyRoles}} {{#message}} -

Persoonlijk bericht van {{displaySenderName}}

-
+

{{{message}}}

{{/message}} {{#useEduID}} -

Voor de toegang tot deze applicatie(s) wordt gebruik gemaakt van eduID. Heb je nog geen eduID account? Geen - probleem, - een eduID account aanmaken kost minder dan 30 seconden.

+

De rol(len) accepteer je door in te loggen met eduID. Als je deze nog niet hebt, kun je hem gelijk aanmaken.

- {{> eduid-logo.svg}} - Uitnodiging accepteren met eduID + style="border-radius: 4px;color: white;display: flex;width: 200px;background-color: #0077c8;text-decoration: none;cursor:pointer;padding: 8px 20px 8px 10px;"> + {{> eduID-logo-square.svg}} + Ga door met eduID
{{/useEduID}} {{^useEduID}} -

Voor de toegang tot deze applicaties wordt gebruik gemaakt van SURFconext.

+

De rol(len) accepteer je door in te loggen met SURFconext.

- Uitnodiging accepteren + style="border-radius: 4px;color: white;display: flex;width: 200px;background-color: #0077c8;text-decoration: none;cursor:pointer;padding: 8px 20px 8px 10px;"> + Ga door
{{/useEduID}} From 666558c162431395a36070d34090647cca7375ab Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Sun, 4 Jan 2026 00:35:54 +0100 Subject: [PATCH 2/8] #549 Improve invite email add url --- server/src/main/resources/templates/invitation_en.html | 2 ++ server/src/main/resources/templates/invitation_nl.html | 2 ++ 2 files changed, 4 insertions(+) diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index 6abf6e27..bd18b427 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -48,6 +48,7 @@ Continue with eduID
+

or copy this link into your browser: {{{ url }}}

{{/useEduID}} {{^useEduID}}

For access to these applications we use SURFconext.

@@ -57,6 +58,7 @@ Continue
+

or copy this link into your browser: {{{ url }}}

{{/useEduID}} {{#useEduID}} diff --git a/server/src/main/resources/templates/invitation_nl.html b/server/src/main/resources/templates/invitation_nl.html index c3fed347..5931adfa 100644 --- a/server/src/main/resources/templates/invitation_nl.html +++ b/server/src/main/resources/templates/invitation_nl.html @@ -48,6 +48,7 @@ Ga door met eduID +

of kopieer deze link in je browser: {{{ url }}}

{{/useEduID}} {{^useEduID}}

De rol(len) accepteer je door in te loggen met SURFconext.

@@ -57,6 +58,7 @@ Ga door +

of kopieer deze link in je browser: {{{ url }}}

{{/useEduID}} {{#useEduID}} From e5ee0e8087b47066ed4ba11d302a5234647a2ccb Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Sun, 4 Jan 2026 00:47:48 +0100 Subject: [PATCH 3/8] #549 Improve invite email fix EN version --- server/src/main/resources/templates/invitation_en.html | 2 +- server/src/test/java/invite/mail/MailBoxTest.java | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index bd18b427..ddb47ab9 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -51,7 +51,7 @@

or copy this link into your browser: {{{ url }}}

{{/useEduID}} {{^useEduID}} -

For access to these applications we use SURFconext.

+

You accept the role(s) by logging in with SURFconext.

diff --git a/server/src/test/java/invite/mail/MailBoxTest.java b/server/src/test/java/invite/mail/MailBoxTest.java index b35258c3..2c075841 100644 --- a/server/src/test/java/invite/mail/MailBoxTest.java +++ b/server/src/test/java/invite/mail/MailBoxTest.java @@ -27,8 +27,8 @@ void sendInviteMail() { assertTrue(htmlContent.contains("Wiki EN")); assertTrue(htmlContent.contains("SURF bv")); - assertTrue(htmlContent.contains("For access to these applications we use SURFconext")); - assertFalse(htmlContent.contains("For access to these applications eduID is used")); + assertTrue(htmlContent.contains("You accept the role(s) by logging in with SURFconext.")); + assertFalse(htmlContent.contains("You accept the role(s) by logging in with eduID")); } @Test @@ -37,8 +37,8 @@ void sendInviteMailForEduIDOnly() { assertTrue(htmlContent.contains("Wiki EN")); assertTrue(htmlContent.contains("SURF bv")); - assertFalse(htmlContent.contains("For access to these applications we use SURFconext")); - assertTrue(htmlContent.contains("For access to these applications eduID is used")); + assertFalse(htmlContent.contains("You accept the role(s) by logging in with SURFconext")); + assertTrue(htmlContent.contains("You accept the role(s) by logging in with eduID")); } private String doSendInviteMail(boolean eduIDOnly, Authority intendedAuthority) { From 1bfbdb1dcb0629c728b3082d9bffd8f36468c654 Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Thu, 8 Jan 2026 21:53:22 +0100 Subject: [PATCH 4/8] #549 Restyle invitation email footer --- .../resources/templates/invitation_en.html | 51 +++++++++---------- .../resources/templates/invitation_nl.html | 51 +++++++++---------- .../resources/templates/logo-surf-black.svg | 4 ++ 3 files changed, 54 insertions(+), 52 deletions(-) create mode 100644 server/src/main/resources/templates/logo-surf-black.svg diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index ddb47ab9..2fd389a2 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -4,12 +4,11 @@ {{title}} -
-
+
+
{{> logo-surf.svg}} - Welcome
-
+
{{#environment}}

This mail is from the {{environment}} environment @@ -33,7 +32,7 @@ {{/invitation.anyRoles}} {{#message}} -

+

{{{message}}}

@@ -48,7 +47,7 @@ Continue with eduID
-

or copy this link into your browser: {{{ url }}}

+

or copy this link into your browser: {{{ url }}}

{{/useEduID}} {{^useEduID}}

You accept the role(s) by logging in with SURFconext.

@@ -58,29 +57,29 @@ Continue
-

or copy this link into your browser: {{{ url }}}

+

or copy this link into your browser: {{{ url }}}

{{/useEduID}}
- {{#useEduID}} -
-

Why do I need an eduID account?

-

An eduID is an account for users within the Dutch educational and research community. It is yours and exists - independent of an educational institution. - With eduID you can log in to various applications. Even if you don't have an institutional account. You can - use - eduID for example - if you want to follow courses with different institutions. Or if you want to follow a course after you have - graduated.

-

- Request eduID -

-

Requesting an eduID is free, and you can do it now. All you need is an - emailaddress.

+
+
+ This invitation is sent via + SURFconext Invite + • + Privacy Policy + • + Terms of Use +
+
+ {{> logo-surf-black.svg}} +
- {{/useEduID}} - - {{> footer_en.html}} -
+ diff --git a/server/src/main/resources/templates/invitation_nl.html b/server/src/main/resources/templates/invitation_nl.html index 5931adfa..393a5246 100644 --- a/server/src/main/resources/templates/invitation_nl.html +++ b/server/src/main/resources/templates/invitation_nl.html @@ -4,12 +4,11 @@ {{title}} -
-
+
+
{{> logo-surf.svg}} - Welkom
-
+
{{#environment}}

Deze mail is van de {{environment}}-omgeving @@ -33,7 +32,7 @@ {{/invitation.anyRoles}} {{#message}} -

+

{{{message}}}

@@ -48,7 +47,7 @@ Ga door met eduID
-

of kopieer deze link in je browser: {{{ url }}}

+

of kopieer deze link in je browser: {{{ url }}}

{{/useEduID}} {{^useEduID}}

De rol(len) accepteer je door in te loggen met SURFconext.

@@ -58,29 +57,29 @@ Ga door
-

of kopieer deze link in je browser: {{{ url }}}

+

of kopieer deze link in je browser: {{{ url }}}

{{/useEduID}}
- {{#useEduID}} -
-

Waarom heb ik een eduID account nodig?

-

- Een eduID is een account voor gebruikers binnen het Nederlandse onderwijs en onderzoek. Het - is van jou, en bestaat onafhankelijk van een onderwijsinstelling. Met eduID kan je inloggen op - verschillende applicaties. Ook als je geen account van je instelling hebt, kan je met eduID inloggen. - Je kunt eduID bijvoorbeeld gebruiken als je bij meerdere instellingen tegelijk onnderwijs wilt volgen. - Of na je studie als je naast je werk een opleiding of cursus wilt volgen. -

-

- Aanvragen eduID -

-

Het aanvragen van een eduID is kosteloos en kan direct. Je hebt alleen een - e-mailadres nodig.

+
+
+ Deze uitnoding verloopt via + SURFconext Invite + • + Privacy Policy + • + Voorwaarden +
+
+ {{> logo-surf-black.svg}} +
- {{/useEduID}} - - {{> footer_nl.html}} -
+ diff --git a/server/src/main/resources/templates/logo-surf-black.svg b/server/src/main/resources/templates/logo-surf-black.svg new file mode 100644 index 00000000..0fdba176 --- /dev/null +++ b/server/src/main/resources/templates/logo-surf-black.svg @@ -0,0 +1,4 @@ + + + + From cf9a608eda1b86b9c874cbe7984a8dbc3c30bac3 Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Thu, 8 Jan 2026 22:57:05 +0100 Subject: [PATCH 5/8] #549 Restyle invitation email --- server/src/main/resources/templates/invitation_en.html | 7 ++++--- server/src/main/resources/templates/invitation_nl.html | 7 ++++--- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index 2fd389a2..7784b1aa 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -4,13 +4,14 @@ {{title}} -
+
{{> logo-surf.svg}}
{{#environment}} -

+

This mail is from the {{environment}} environment

{{/environment}} @@ -61,7 +62,7 @@ {{/useEduID}}
+ style="display:flex;align-items:center;justify-content:space-between;padding: 12px 0;line-height: 24px; color: #5E6873; border-top: 1px solid #888F99;font-size: 14px;">
This invitation is sent via {{title}} -
+
{{> logo-surf.svg}}
{{#environment}} -

+

Deze mail is van de {{environment}}-omgeving

{{/environment}} @@ -61,7 +62,7 @@ {{/useEduID}}
+ style="display:flex;align-items:center;justify-content:space-between;padding: 12px 0;line-height: 24px; color: #5E6873; border-top: 1px solid #888F99;font-size: 14px;">
Deze uitnoding verloopt via Date: Fri, 9 Jan 2026 16:28:36 +0100 Subject: [PATCH 6/8] #549 Restyle invitation email --- server/src/main/java/invite/mail/MailBox.java | 5 +++ .../resources/templates/invitation_en.html | 23 ++++++----- .../resources/templates/invitation_nl.html | 23 ++++++----- .../invite/api/InvitationControllerTest.java | 41 +++++++++++++++++++ .../api/InvitationMailControllerTest.java | 1 - .../InternalInviteControllerTest.java | 33 +++++++++++++++ .../test/java/invite/mail/MailBoxTest.java | 2 - 7 files changed, 103 insertions(+), 25 deletions(-) diff --git a/server/src/main/java/invite/mail/MailBox.java b/server/src/main/java/invite/mail/MailBox.java index 1e0bda77..6bffc13a 100644 --- a/server/src/main/java/invite/mail/MailBox.java +++ b/server/src/main/java/invite/mail/MailBox.java @@ -73,8 +73,13 @@ public void sendInviteMail(Provisionable provisionable, Invitation invitation, .getIdentityProvider(user.getSchacHomeOrganization()) .map(idp -> idp.getName()) .orElse(user.getSchacHomeOrganization())); + variables.put("institutionLogoUrl", idPMetaDataResolver + .getIdentityProvider(user.getSchacHomeOrganization()) + .map(idp -> idp.getLogoUrl()) + .orElse(null)); } else { variables.put("institutionName", "SURF"); + variables.put("isInstitutionSurf", true); } optionalIdpName.ifPresent(idpName -> variables.put("idpName", idpName)); variables.put("roles", splitListSemantically(invitation.getRoles().stream() diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index 7784b1aa..aa60b241 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -4,10 +4,14 @@ {{title}} -
+
- {{> logo-surf.svg}} + {{#institutionLogoUrl}} + logo + {{/institutionLogoUrl}} + {{#isInstitutionSurf}} + logo + {{/isInstitutionSurf}}
{{#environment}} @@ -15,20 +19,17 @@ This mail is from the {{environment}} environment

{{/environment}} -

Hello,

- {{#invitation.institutionAdminInvitation}} -

{{displaySenderName}} (from {{institutionName}}) has invited you to become +

{{displaySenderName}} (from {{institutionName}}) has invited you to become an institution admin for institution {{idpName}} to manage the applications of this institution.

{{/invitation.institutionAdminInvitation}} {{#invitation.anyRoles}} -

{{displaySenderName}} (from {{institutionName}}) would you like to grant access to:

+

{{displaySenderName}} (from {{institutionName}}) would you like to grant access to:

    {{#groupedProviders}} -
  • {{name}} ({{organisation}}) - as - {{rolesDisplay}}
    • +
  • {{name}} - as {{rolesDisplay}}
    • {{/groupedProviders}}
{{/invitation.anyRoles}} @@ -43,7 +44,7 @@

You accept the role(s) by logging in with eduID. If you don’t have one yet, you can create it right away.

+ style="border-radius: 4px;color: white;display: flex;background-color: #0077c8;text-decoration: none;cursor:pointer;padding: 8px 20px 8px 10px;"> {{> eduID-logo-square.svg}} Continue with eduID @@ -54,7 +55,7 @@

You accept the role(s) by logging in with SURFconext.

diff --git a/server/src/main/resources/templates/invitation_nl.html b/server/src/main/resources/templates/invitation_nl.html index f2fb97e2..f4be4f77 100644 --- a/server/src/main/resources/templates/invitation_nl.html +++ b/server/src/main/resources/templates/invitation_nl.html @@ -4,10 +4,14 @@ {{title}} -
+
- {{> logo-surf.svg}} + {{#institutionLogoUrl}} + logo + {{/institutionLogoUrl}} + {{#isInstitutionSurf}} + logo + {{/isInstitutionSurf}}
{{#environment}} @@ -15,20 +19,17 @@ Deze mail is van de {{environment}}-omgeving

{{/environment}} -

Hallo,

- {{#invitation.institutionAdminInvitation}} -

{{displaySenderName}} (van {{institutionName}}) heeft je uitgenodigd om +

{{displaySenderName}} (van {{institutionName}}) heeft je uitgenodigd om een instellings-admin te worden van instelling {{idpName}} om de applicaties van deze instelling te beheren.

{{/invitation.institutionAdminInvitation}} {{#invitation.anyRoles}} -

{{displaySenderName}} (van {{institutionName}}) wil je toegang geven tot:

+

{{displaySenderName}} (van {{institutionName}}) wil je toegang geven tot:

    {{#groupedProviders}} -
  • {{name}} ({{organisation}}) - als - {{rolesDisplay}}
    • +
  • {{name}} - als {{rolesDisplay}}
    • {{/groupedProviders}}
{{/invitation.anyRoles}} @@ -43,7 +44,7 @@

De rol(len) accepteer je door in te loggen met eduID. Als je deze nog niet hebt, kun je hem gelijk aanmaken.

+ style="border-radius: 4px;color: white;display: flex;background-color: #0077c8;text-decoration: none;cursor:pointer;padding: 8px 20px 8px 10px;"> {{> eduID-logo-square.svg}} Ga door met eduID @@ -54,7 +55,7 @@

De rol(len) accepteer je door in te loggen met SURFconext.

diff --git a/server/src/test/java/invite/api/InvitationControllerTest.java b/server/src/test/java/invite/api/InvitationControllerTest.java index 312da87b..dd6446db 100644 --- a/server/src/test/java/invite/api/InvitationControllerTest.java +++ b/server/src/test/java/invite/api/InvitationControllerTest.java @@ -107,6 +107,47 @@ void newInvitation() throws Exception { assertEquals(2, ((List) results.get("recipientInvitationURLs")).size()); } + @Test + void newInvitationNL() throws Exception { + //Because the user is changed and provisionings are queried + stubForManageProvisioning(List.of()); + AccessCookieFilter accessCookieFilter = openIDConnectFlow("/api/v1/users/login", MANAGE_SUB); + + stubForManageProviderById(EntityType.SAML20_SP, "1"); + + List roleIdentifiers = roleRepository.findByApplicationUsagesApplicationManageId("1").stream() + .map(Role::getId) + .toList(); + InvitationRequest invitationRequest = new InvitationRequest( + Authority.GUEST, + "Message", + Language.nl, + true, + false, + null, + false, + false, + List.of("new@new.nl"), + List.of(new Invite("new2@new.nl", "placeholder-1")), + roleIdentifiers, + null, + Instant.now().plus(365, ChronoUnit.DAYS), + Instant.now().plus(12, ChronoUnit.DAYS)); + + Map results = given() + .when() + .filter(accessCookieFilter.cookieFilter()) + .accept(ContentType.JSON) + .header(accessCookieFilter.csrfToken().getHeaderName(), accessCookieFilter.csrfToken().getToken()) + .contentType(ContentType.JSON) + .body(invitationRequest) + .post("/api/v1/invitations") + .as(new TypeRef<>() { + }); + assertEquals(201, results.get("status")); + assertEquals(2, ((List) results.get("recipientInvitationURLs")).size()); + } + @Test void newInvitationEmptyRoles() throws Exception { //Because the user is changed and provisionings are queried diff --git a/server/src/test/java/invite/api/InvitationMailControllerTest.java b/server/src/test/java/invite/api/InvitationMailControllerTest.java index 17424954..85090b74 100644 --- a/server/src/test/java/invite/api/InvitationMailControllerTest.java +++ b/server/src/test/java/invite/api/InvitationMailControllerTest.java @@ -34,7 +34,6 @@ void resendInviteMail() throws Exception { .statusCode(201); String htmlContent = super.mailMessage().getHtmlContent(); assertTrue(htmlContent.contains("Calendar EN")); - assertTrue(htmlContent.contains("SURF bv")); assertTrue(htmlContent.contains("Mail")); } diff --git a/server/src/test/java/invite/internal/InternalInviteControllerTest.java b/server/src/test/java/invite/internal/InternalInviteControllerTest.java index 933ab702..737a9437 100644 --- a/server/src/test/java/invite/internal/InternalInviteControllerTest.java +++ b/server/src/test/java/invite/internal/InternalInviteControllerTest.java @@ -157,6 +157,39 @@ void newInvitation() { assertEquals(201, results.get("status")); assertEquals(1, ((List) results.get("recipientInvitationURLs")).size()); } + @Test + void newInvitationNL() { + stubForManageProviderById(EntityType.SAML20_SP, "4"); + List roleIdentifiers = List.of(roleRepository.findByName("Research").get().getId()); + + InvitationRequest invitationRequest = new InvitationRequest( + Authority.GUEST, + "Message", + Language.nl, + true, + true, + null, + false, + false, + List.of("new@new.nl"), + List.of(), + roleIdentifiers, + null, + Instant.now().plus(365, ChronoUnit.DAYS), + Instant.now().plus(12, ChronoUnit.DAYS)); + + Map results = given() + .when() + .auth().preemptive().basic("sp_dashboard", "secret") + .accept(ContentType.JSON) + .contentType(ContentType.JSON) + .body(invitationRequest) + .post("/api/internal/invite/invitations") + .as(new TypeRef>() { + }); + assertEquals(201, results.get("status")); + assertEquals(1, ((List) results.get("recipientInvitationURLs")).size()); + } @Test void resendInvitation() { diff --git a/server/src/test/java/invite/mail/MailBoxTest.java b/server/src/test/java/invite/mail/MailBoxTest.java index 2c075841..6e232280 100644 --- a/server/src/test/java/invite/mail/MailBoxTest.java +++ b/server/src/test/java/invite/mail/MailBoxTest.java @@ -26,7 +26,6 @@ void sendInviteMail() { String htmlContent = doSendInviteMail(true, Authority.INVITER); assertTrue(htmlContent.contains("Wiki EN")); - assertTrue(htmlContent.contains("SURF bv")); assertTrue(htmlContent.contains("You accept the role(s) by logging in with SURFconext.")); assertFalse(htmlContent.contains("You accept the role(s) by logging in with eduID")); } @@ -36,7 +35,6 @@ void sendInviteMailForEduIDOnly() { String htmlContent = doSendInviteMail(true, Authority.GUEST); assertTrue(htmlContent.contains("Wiki EN")); - assertTrue(htmlContent.contains("SURF bv")); assertFalse(htmlContent.contains("You accept the role(s) by logging in with SURFconext")); assertTrue(htmlContent.contains("You accept the role(s) by logging in with eduID")); } From 06667a5198b1ea0b092dd9864e8c793608c752a8 Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Fri, 9 Jan 2026 16:31:22 +0100 Subject: [PATCH 7/8] #549 Restyle invitation email --- server/src/main/resources/templates/invitation_en.html | 2 +- server/src/main/resources/templates/invitation_nl.html | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/server/src/main/resources/templates/invitation_en.html b/server/src/main/resources/templates/invitation_en.html index aa60b241..12958632 100644 --- a/server/src/main/resources/templates/invitation_en.html +++ b/server/src/main/resources/templates/invitation_en.html @@ -4,7 +4,7 @@ {{title}} -
+
{{#institutionLogoUrl}} logo diff --git a/server/src/main/resources/templates/invitation_nl.html b/server/src/main/resources/templates/invitation_nl.html index f4be4f77..33efc568 100644 --- a/server/src/main/resources/templates/invitation_nl.html +++ b/server/src/main/resources/templates/invitation_nl.html @@ -4,7 +4,7 @@ {{title}} -
+
{{#institutionLogoUrl}} logo From 08274a62011bbf711b09e8de27a3916bf96a52e4 Mon Sep 17 00:00:00 2001 From: Leroy <3416288+Liemine@users.noreply.github.com> Date: Fri, 9 Jan 2026 16:41:20 +0100 Subject: [PATCH 8/8] #549 Restyle invitation email --- server/src/main/java/invite/mail/MailBox.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/server/src/main/java/invite/mail/MailBox.java b/server/src/main/java/invite/mail/MailBox.java index 6bffc13a..3b7c940a 100644 --- a/server/src/main/java/invite/mail/MailBox.java +++ b/server/src/main/java/invite/mail/MailBox.java @@ -1,6 +1,7 @@ package invite.mail; import invite.cron.IdPMetaDataResolver; +import invite.cron.IdentityProvider; import invite.model.*; import com.fasterxml.jackson.core.type.TypeReference; import com.fasterxml.jackson.databind.ObjectMapper; @@ -69,12 +70,12 @@ public void sendInviteMail(Provisionable provisionable, Invitation invitation, variables.put("groupedProviders", groupedProviders); variables.put("title", title); if (provisionable instanceof User user) { - variables.put("institutionName", idPMetaDataResolver - .getIdentityProvider(user.getSchacHomeOrganization()) + Optional identityProvider = idPMetaDataResolver + .getIdentityProvider(user.getSchacHomeOrganization()); + variables.put("institutionName", identityProvider .map(idp -> idp.getName()) .orElse(user.getSchacHomeOrganization())); - variables.put("institutionLogoUrl", idPMetaDataResolver - .getIdentityProvider(user.getSchacHomeOrganization()) + variables.put("institutionLogoUrl", identityProvider .map(idp -> idp.getLogoUrl()) .orElse(null)); } else {