Skip to content

Changelog for expert update banner#4608

Merged
Steve-Mcl merged 6 commits intomainfrom
4594-changelog-in-app-banner-for-update
Mar 5, 2026
Merged

Changelog for expert update banner#4608
Steve-Mcl merged 6 commits intomainfrom
4594-changelog-in-app-banner-for-update

Conversation

@Steve-Mcl
Copy link
Contributor

@Steve-Mcl Steve-Mcl commented Feb 27, 2026

Description

Changelog for expert update banner

IMPORTANT: Do not merge until all other tasks in the parent issue tracker are complete

Related Issue(s)

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
  • For blog PRs, an Art Request has been created (instructions)

@netlify
Copy link

netlify bot commented Feb 27, 2026

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit b6ca915
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/69a9c3cb4d0ae50007856ce6
😎 Deploy Preview https://deploy-preview-4608--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 90 (🔴 down 3 from production)
Accessibility: 81 (no change from production)
Best Practices: 100 (no change from production)
SEO: 91 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@dimitrieh dimitrieh left a comment

Choose a reason for hiding this comment

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

Superseded — see suggestion comments below.

Copy link
Contributor

@dimitrieh dimitrieh left a comment

Choose a reason for hiding this comment

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

Reviewed against brand voice guidelines — lead with user pain point, remove filler, end with confidence.

Copy link
Contributor

@dimitrieh dimitrieh left a comment

Choose a reason for hiding this comment

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

@Steve-Mcl see if you agree with the above :)

@dimitrieh
Copy link
Contributor

@Steve-Mcl Had another look, think the above is good and a play on your words. Think our definition of changelog writing is something to add to our handbook. A thing on my list to do.

Steve-Mcl and others added 2 commits March 5, 2026 13:26
Co-authored-by: Dimitrie Hoekstra <dimitrie@flowfuse.com>
Co-authored-by: Dimitrie Hoekstra <dimitrie@flowfuse.com>
@Steve-Mcl
Copy link
Contributor Author

@dimitrieh reasonalbe suggestions (this time ;) )

Please re-review.

@Steve-Mcl
Copy link
Contributor Author

@dimitrieh this feature was pushed to main / live yesterday 12:16 - it would be good if you can approve/merge please.

Copy link
Contributor

@dimitrieh dimitrieh left a comment

Choose a reason for hiding this comment

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

let's go

@Steve-Mcl Steve-Mcl enabled auto-merge March 5, 2026 16:08
@Steve-Mcl Steve-Mcl merged commit 714f1ee into main Mar 5, 2026
5 checks passed
@Steve-Mcl Steve-Mcl deleted the 4594-changelog-in-app-banner-for-update branch March 5, 2026 18:00
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 in app banner to let user that new features are available in update

3 participants