Skip to content

BE: preheader + typo#1712

Merged
jokob-sk merged 1 commit into
mainfrom
next_release
Jul 13, 2026
Merged

BE: preheader + typo#1712
jokob-sk merged 1 commit into
mainfrom
next_release

Conversation

@jokob-sk

@jokob-sk jokob-sk commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Bug Fixes

    • Corrected the English description for the notification delay setting.
    • Improved hidden email preheader formatting to prevent unintended preview text across email clients.
  • Style

    • Updated invisible preheader spacing in generated email reports while preserving the displayed content.

@coderabbitai

coderabbitai Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 67e498e4-cd56-47e7-b60c-33c034a11200

📥 Commits

Reviewing files that changed from the base of the PR and between 1f3e62b and 62bcb07.

📒 Files selected for processing (3)
  • front/plugins/_publisher_mqtt/config.json
  • front/report_templates/report_template.html
  • server/models/notification_instance.py

📝 Walkthrough

Walkthrough

The changes correct an MQTT setting description and update email preheader hiding styles and padding generation.

Changes

Email preheader updates

Layer / File(s) Summary
Preheader markup and padding
front/report_templates/report_template.html, server/models/notification_instance.py
The hidden preheader span uses expanded inline CSS, while notification email padding uses the new repeated ‌ and   entity pattern.

MQTT localization

Layer / File(s) Summary
MQTT delay description
front/plugins/_publisher_mqtt/config.json
The English DELAY_SEC description corrects a typo about subsequent scans and notifications.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the two main changes: a preheader update and a typo fix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch next_release

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jokob-sk jokob-sk merged commit 2924b4c into main Jul 13, 2026
7 checks passed
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.

1 participant