Skip to content

Add FlowFuse 'Try It Now' CTA to Node-RED page #3379

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

Merged
merged 5 commits into from
Jul 14, 2025
Merged

Conversation

gstout52
Copy link
Contributor

@gstout52 gstout52 commented Jun 18, 2025

Description

Changed CTA on Node-RED page to point to cloud signup.

Related Issue(s)

n/a

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

@gstout52 gstout52 requested a review from ZJvandeWeg June 18, 2025 22:04
@ZJvandeWeg ZJvandeWeg requested a review from Yndira-E June 19, 2025 05:06
@Yndira-E
Copy link
Contributor

@gstout52 a couple of thoughts:

  1. Adding this extra line in the hero section makes the content feel a bit too long and throws off the visual balance of the design.
  2. More importantly, I’m not sure it’s ideal to include an external link in the hero section. We already have a primary CTA there, and best practice is typically to keep it focused. Directing users away from our site that early might reduce engagement with the rest of our content.

Maybe there's another place we could surface that link without compromising the layout or flow of the page? Also worth noting: the PDF from the CTA already includes links to the official Node-RED docs https://nodered.org/docs/* with guidance on getting started.

@gstout52 gstout52 marked this pull request as ready for review June 20, 2025 13:48
@gstout52 gstout52 marked this pull request as draft June 20, 2025 14:38
- Update Node-RED page CTA from 'Download it now' to 'Try It Now'
- Point CTA to FlowFuse signup URL ({{ site.appURL }}/account/create)
- Add analytics tracking for 'cta-try-it-now' event
- Align with Free Trial button functionality in top navigation
@gstout52 gstout52 changed the title Added Node-RED link Add FlowFuse 'Try It Now' CTA to Node-RED page Jul 3, 2025
- Remove 'Check out our free guide...' promotional text
- Remove 'Download eBook' CTA buttons (desktop and mobile)
- Convert 'Try It Now' text link to primary CTA button
- Maintain consistent FlowFuse signup URL and analytics tracking
- Simplify hero section messaging to focus on getting started
@gstout52 gstout52 marked this pull request as ready for review July 3, 2025 21:11
@ZJvandeWeg
Copy link
Member

@gstout52 I would ego @Yndira-E here concerns, but would like to offer a suggestion; what about having a section that installs the device agent? Which offers a copy+paste of the agent? What's the recommended onboarding for them?

If I'm a NR user, I'd expect a one liner to get me started.

@gstout52
Copy link
Contributor Author

@gstout52 I would ego @Yndira-E here concerns, but would like to offer a suggestion; what about having a section that installs the device agent? Which offers a copy+paste of the agent? What's the recommended onboarding for them?

If I'm a NR user, I'd expect a one liner to get me started.

Users installing the Device Agent are given a single line of code during the installation process.

@gstout52 gstout52 requested a review from Yndira-E July 10, 2025 19:23
</div>
</div>
<!--Hero Content-->
<div class="w-full pt-12 pb-20 md:pt-6 md:pb-12">
Copy link
Member

Choose a reason for hiding this comment

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

@gstout52 can we not redo the denting and have a huge diff? I'm confused now about what changed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ZJvandeWeg Claude helped me adjust the indentations. Does this look cleaner?

- Restore proper indentation for HTML structure
- Addresses ZJ's comment about large diff due to indentation changes
- Maintains same functionality while reducing diff size

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ZJvandeWeg ZJvandeWeg merged commit 8241396 into main Jul 14, 2025
5 checks passed
@ZJvandeWeg ZJvandeWeg deleted the node-red-page-edit branch July 14, 2025 15:08
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.

3 participants