action #111215
closedQA (public) - coordination #99303: [saga][epic] Future improvements for SUSE Maintenance QA workflows with fully automated testing, approval and release
coordination #99306: [epic] Future improvements: Make reviewing openQA results per squad easier
Various improvements for email notification about unreviewed jobs size:M
Description
Observations¶
- Expanding a message creates a lot of vertical white space.
- The job group ID is not very useful - better use the group name
- The email sender name is quite long, it could be shorter, e.g. "OSD review"
- Links are not clickable
Acceptance criteria¶
- AC1: URLs are clickable
- AC2: Human-readable group names are visible
- AC3: Retain plain text email content
Suggestions¶
- This seems to be a Slack problem - the original emails don't have extra trailing lines
- Should be very easy
- Try if URLs can be clickable in subject or if there is any formatting other than HTML that would work
- We need to send HTML emails instead, then links are clickable (tested)
- See also #111135
- Send a multi-part email with markdown and HTML
Files
Updated by tinita over 2 years ago
- Related to action #111135: Enhance email notification message content for about failed and unreviewed jobs added
Updated by okurz over 2 years ago
I would be happy if we could avoid sending HTML emails. Maybe there is a special format, maybe markdown?, that would make links clickable? Maybe if the URL is even in the subject line?
Updated by mkittler over 2 years ago
- Subject changed from Various improvements for email notification about unreviewed jobs to Various improvements for email notification about unreviewed jobs size:M
- Description updated (diff)
- Status changed from New to Workable
Updated by okurz over 2 years ago
I found that in my Android Slack client the URLs are already clickable, see
Updated by tinita over 2 years ago
- Status changed from Workable to In Progress
- Assignee set to tinita
Updated by tinita over 2 years ago
https://github.com/os-autoinst/scripts/pull/164 Add dependency Text::Markdown (merged)
https://github.com/os-autoinst/os-autoinst-common/pull/21 Add test for make update-deps (merged)
https://github.com/os-autoinst/scripts/pull/165 Add test for make update-deps (merged)
Updated by tinita over 2 years ago
https://github.com/os-autoinst/openQA/pull/4675 Allow markdown in apparmor (merged)
Updated by tinita over 2 years ago
https://github.com/os-autoinst/scripts/pull/166 Send multipart email to improve formatting
Still blocked by the above PRs
Updated by tinita over 2 years ago
- Status changed from In Progress to Feedback
https://github.com/os-autoinst/scripts/pull/166 merged
I added a trigger to automatically rebuild https://build.opensuse.org/package/show/devel:openQA/os-autoinst-scripts on push, which updates os-autoinst-scripts-deps
Updated by tinita over 2 years ago
https://gitlab.suse.de/openqa/salt-states-openqa/-/merge_requests/689 by @okurz (for installing/updating os-autoinst-scripts-deps on osd) was merged.
Updated by tinita over 2 years ago
- Status changed from Feedback to Resolved
Deployed and everything looks good so far.