Skip to content

Add Documentation For Ethernet IP node#5249

Open
sumitshinde-84 wants to merge 4 commits into
mainfrom
ethernet-ip-documentation
Open

Add Documentation For Ethernet IP node#5249
sumitshinde-84 wants to merge 4 commits into
mainfrom
ethernet-ip-documentation

Conversation

@sumitshinde-84

@sumitshinde-84 sumitshinde-84 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Description

This PR creates documentation for the Ethernet/IP node. The content is largely based on the README, with the goal of making it publicly available documentation . I also added some notes and made a few edits, such as removing Node-RED references and updating the package name to remove the -nr prefix.

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

netlify Bot commented Jun 25, 2026

Copy link
Copy Markdown

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit 0c1fbf7
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/6a3e9a0084b2e40008a54f68
😎 Deploy Preview https://deploy-preview-5249--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: 64 (🔴 down 3 from production)
Accessibility: 96 (no change from production)
Best Practices: 100 (no change from production)
SEO: 83 (🔴 down 8 from production)
PWA: -
View the detailed breakdown and full score reports

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

@hardillb hardillb left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We have chosen to call this node @flowfuse-certified-nodes/cip-suite, do we want to have CIP-Suite as the title rather than Ethernet-IP?

@sumitshinde-84

Copy link
Copy Markdown
Contributor Author

Updated the package name to @flowfuse-certified-nodes/cip-suite and URL to /cip-suite. For the title I used "CIP Suite — EtherNet/IP Node" rather than just "CIP Suite", keeping ethernet ip in the title helps discoverability.

@sumitshinde-84

Copy link
Copy Markdown
Contributor Author

@hardillb what do you think ?

Comment on lines +79 to +86
### Install via npm

If you manage your palette from the command line, install the package directly:

```bash
npm install @flowfuse-certified-nodes/cip-suite
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
### Install via npm
If you manage your palette from the command line, install the package directly:
```bash
npm install @flowfuse-certified-nodes/cip-suite
```

Remove all of this as you can't install it from command line as you will not have the required npm tokens and config. This applies to ALL certified nodes

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.

2 participants