Skip to content

Conversation

@omrid01
Copy link
Contributor

@omrid01 omrid01 commented Aug 24, 2025

Description

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
    • Upgrade instructions
    • Configuration details
    • Concepts
  • Changes flowforge.yml?
    • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
    • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
  • Link to Changelog Entry PR, or note why one is not needed.

Labels

  • Includes a DB migration? -> add the area:migration label

@omrid01 omrid01 changed the title Configuring Noder-RED dashboard 2.0 on Home Assistant Configuring Node-RED dashboard 2.0 on Home Assistant Aug 24, 2025
@joepavitt joepavitt self-requested a review September 8, 2025 10:38
@joepavitt
Copy link
Contributor

joepavitt commented Sep 8, 2025

@omrid01 can you upload the images you've used to comments in this thread please, they're not loading when running the docs locally, or in production as the images you've linked to are private to your repository. Or, even better, upload them into the /docs/assets/images folder and then update the references in the docs to load them.

I've also got a bunch of changes which I have just pushed.

Copy link
Contributor

@Steve-Mcl Steve-Mcl left a comment

Choose a reason for hiding this comment

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

Hi @omrid01 - I would like to get this merged. Would you be kind enough to address the issues raised in review please?

2. Create a new iframe container for hosting the FlowFuse Dashboard clients:
- In HA, go to **Settings->Dashboards->Add dashboard**, select **Webpage**

![Image](https://github.com/user-attachments/assets/6b6323f1-e3a3-41d2-8893-c4a79a2cb341)
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add the image to docs/assets/images and point this to it like this:

![A note to what this image is](../../../assets/images/my-image-file-name.png){data-zoomable}
*A note to what this image is*


- Set the URL of the dashboard, e.g. `<HA host>:1880/endpoint/dashboard`

![Image](https://github.com/user-attachments/assets/9815d378-ca6c-458c-8de4-33b40934bd05)
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add the image to docs/assets/images and point this to it like this:

![A note to what this image is](../../../assets/images/my-image-file-name.png){data-zoomable}
*A note to what this image is*

@omrid01
Copy link
Contributor Author

omrid01 commented Oct 5, 2025 via email

@omrid01
Copy link
Contributor Author

omrid01 commented Oct 5, 2025

I have updated the document with locally placed images. Is there anything else I need to do, or do the updates in the fork automatically propagate into the Pull Request?

@Steve-Mcl Steve-Mcl self-requested a review October 6, 2025 13:50
Copy link
Contributor

@Steve-Mcl Steve-Mcl left a comment

Choose a reason for hiding this comment

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

LGTM.
Thank you very much for the docs update.

@Steve-Mcl Steve-Mcl merged commit 0435d6b into FlowFuse:main Oct 6, 2025
1 of 2 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.

3 participants