Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

changelog for notification improvements #2504

Merged
merged 3 commits into from
Aug 28, 2024

Conversation

Steve-Mcl
Copy link
Contributor

@Steve-Mcl Steve-Mcl commented Aug 27, 2024

closes #2503

Do not merge until after FlowFuse/flowfuse#4409 is merged

Description

adds changelog entry for FlowFuse/flowfuse#4295

Related Issue(s)

owner #2503
parent FlowFuse/flowfuse#4408

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated

@Steve-Mcl Steve-Mcl linked an issue Aug 27, 2024 that may be closed by this pull request
Copy link
Contributor

Images automagically compressed by Calibre's image-actions

Compression reduced images by 15.3%, saving 10.81 KB.

Filename Before After Improvement Visual comparison
src/changelog/2024/07/images/notifications-inbox-read.png 40.80 KB 35.00 KB -14.2% View diff
src/changelog/2024/07/images/notifications-inbox-unread.png 29.92 KB 24.92 KB -16.7% View diff

1332 images did not require optimisation.

Co-authored-by: Joe Pavitt <99246719+joepavitt@users.noreply.github.com>
@joepavitt joepavitt merged commit abf7f1a into main Aug 28, 2024
6 checks passed
@joepavitt joepavitt deleted the 2503-changelog-for-instance-crash-notifications branch August 28, 2024 11:11
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.

Changelog for instance crash notifications
2 participants