Skip to content

feat(notifier): render portal URL as clickable hyperlink in email body#105

Open
anup-kodlekere wants to merge 1 commit into
IBM:mainfrom
anup-kodlekere:mail_href
Open

feat(notifier): render portal URL as clickable hyperlink in email body#105
anup-kodlekere wants to merge 1 commit into
IBM:mainfrom
anup-kodlekere:mail_href

Conversation

@anup-kodlekere

Copy link
Copy Markdown
Member

This PR introduces a PORTAL_URL environment variable to embed a clickable hyperlink to the IBM® Power® Access Cloud portal within the email body. The email template has been migrated from plain text to HTML, and the mail content type has been updated accordingly.

With this update, the event-notifier also checks for PORTAL_URL env var to be available at event-notifier startup.

Closes #84

- Switch email body template from plain text to HTML
- Add <a href> anchor around "IBM® Power® Access Cloud" portal text
- Read portal URL from PORTAL_URL env var
- Validate PORTAL_URL is set on event-notifier startup

Signed-off-by: anup-kodlekere <anup.kodlekere@ibm.com>
@anup-kodlekere
anup-kodlekere marked this pull request as ready for review July 1, 2026 11:25
Generated at: {{ .Event.CreatedAt.Format "Jan 02, 2006 15:04:05 UTC" }}
<br><br>
Thanks,<br>
IBM® Power®; Access Cloud Team.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this should be IBM® Power® Access Cloud Team right

@vikasbolla

Copy link
Copy Markdown
Contributor

Thanks for the PR @anup-kodlekere . But this needs to be validated to see if the formatting works properly or not in the email body. And also in out last testing we saw that the url will be huge as it might need to be whitelisted from security team for that to be displayed as IBM® Power® Access Cloud portal.

@anup-kodlekere

Copy link
Copy Markdown
Member Author

Okay, thanks for the context. I cannot comment about the security team's assertions but I can certainly test this (with a minimal reproducible instead of deploying entire PAC) and update the results here. Does that sound good?

@vikasbolla

Copy link
Copy Markdown
Contributor

Okay, thanks for the context. I cannot comment about the security team's assertions but I can certainly test this (with a minimal reproducible instead of deploying entire PAC) and update the results here. Does that sound good?

Sure, you can try that

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Email] Include IBM® Power® Access Cloud as an hyperlink in the email sent to user.

2 participants