Skip to content

[stack] feat(mail): highlight warnings in account security emails - #65222

Open
skjnldsv wants to merge 1 commit into
feature/dav-mail-blocksfrom
feature/security-mail-notes
Open

skjnldsv wants to merge 1 commit into
feature/dav-mail-blocksfrom
feature/security-mail-notes

Conversation

@skjnldsv

@skjnldsv skjnldsv commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Summary

Builds on #65221. The sentence that matters in security mails now stands out in a warning or info box instead of being buried in the text.

Mail Change
Remote wipe started / finished Warning / info box
Password changed "If you did not request this…" in a warning box
Email changed Same warning box, plus a card with the old and new address
Code integrity changed The findings in a warning box

New strings: "Previous email address", "New email address".

Before After (light) After (dark) Mobile
Remote wipe started
Remote wipe finished
Password changed
Email changed
Code integrity

Browser renders of the real code with sample data, after includes the restyle from #65194. Not checked in mail clients.

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests are included
  • Screenshots before/after for front-end changes
  • Documentation has been updated or is not required
  • Backports requested where applicable
  • Labels added where applicable
  • Milestone added for target branch/version

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

👾 This pull request was assisted by Claude Code, commits carry an Assisted-by trailer.

@skjnldsv skjnldsv added this to the Nextcloud 36 milestone Oct 6, 2026
@skjnldsv skjnldsv self-assigned this Oct 6, 2026
@skjnldsv
skjnldsv force-pushed the feature/security-mail-notes branch from ab97722 to 418e03f Compare October 6, 2026 18:15
@skjnldsv
skjnldsv force-pushed the feature/security-mail-notes branch 2 times, most recently from 418e03f to ddd7889 Compare October 6, 2026 18:23
@skjnldsv skjnldsv changed the title feat(mail): highlight warnings in account security emails [stack] feat(mail): highlight warnings in account security emails Oct 6, 2026
@skjnldsv
skjnldsv changed the base branch from feature/mail-template-blocks to feature/dav-mail-blocks October 6, 2026 18:24
@skjnldsv
skjnldsv added this pull request to stack #65223 October 6, 2026 18:34
@skjnldsv
skjnldsv marked this pull request as ready for review October 6, 2026 18:35
@skjnldsv
skjnldsv requested a review from a team as a code owner October 6, 2026 18:35
@skjnldsv skjnldsv removed the 2. developing Work in progress label Oct 6, 2026
@skjnldsv
skjnldsv requested review from CarlSchwan, come-nc, leftybournes and provokateurin and removed request for a team October 6, 2026 18:35
@skjnldsv skjnldsv added the 3. to review Waiting for reviews label Oct 6, 2026
@skjnldsv
skjnldsv force-pushed the feature/security-mail-notes branch from ddd7889 to 903079e Compare October 6, 2026 20:27
@skjnldsv
skjnldsv force-pushed the feature/security-mail-notes branch from 903079e to 4e6b2a0 Compare October 7, 2026 08:25
Put the key sentence of the remote wipe, password changed, email changed
and code integrity emails into a warning or info note. The email changed
mail also shows the previous and new address in a details card.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: John Molakvoæ <14975046+skjnldsv@users.noreply.github.com>

This branch has not been deployed

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant