Feat/xxx/ifm barcode nodered #82
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Deploying with
|
| Latest commit: |
8629a7e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9acabb28.umh-docs-umh-app.pages.dev |
| Branch Preview URL: | https://feat-xxx-ifm-barcode-nodered.umh-docs-umh-app.pages.dev |
…to feat/xxx/ifm_barcode_nodered
JeremyTheocharis
changed the base branch from
main
to
feat/xxx/features-section
JeremyTheocharis
requested changes
Mar 14, 2023
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/barcodereader-retrofitting/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/barcodereader-retrofitting/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/barcodereader-retrofitting/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/barcodereader-retrofitting/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/barcodereader-retrofitting/index.md
Outdated
Show resolved
Hide resolved
|
|
change: first sentence, less repetitive add: Step by Step instruction for the barcode reader
add: -Step by Step instruction for sensorconnect -Links to ifm and the retrofitting blog article -info for analog and digital sensors
JeremyTheocharis
requested changes
Apr 24, 2023
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
DaruZero
suggested changes
Apr 25, 2023
There was a problem hiding this comment.
Why are the new articles in page bundles if it only contains one .md file? Please convert to standalone pages.
From this
features/
├── barcodereader-retrofitting/
│ └── index.md
├── data-connectivity-node-red/
│ └── index.md
...
to this
features/
├── barcodereader-retrofitting.md
├── data-connectivity-node-red.md
...
Also. please use the correct formatting style.
- Each line should not be longer than 80 characters. You can set a vertical line in your text editor to help you with that. Depending on the editor you use, the setting might be called ruler, right margin, or column
- Use a markdown linter. For example, in VS code there is an extension called markdownlint that will show you common style errors, like not having a blank line before and after a code block
``` ... ```
Lastly, remember to squash the commits before merging. Here more info about that. Ask me for help if you have doubts
umh.docs.umh.app/content/en/docs/features/barcodereader-retrofitting/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
|
|
||
| ## When should I use it? | ||
|
|
||
| Sometimes it is necessary to connect a lot of different protocols (e.g Siemens-S7, OPC-UA, Serial, ...) and node-RED can be a maintainable |
There was a problem hiding this comment.
There was a problem hiding this comment.
dont get that point. I didnt use "We" _(.)_/
There was a problem hiding this comment.
I was talking about the "e.g", but it's too late now I guess
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/data-connectivity-node-red/index.md
Outdated
Show resolved
Hide resolved
umh.docs.umh.app/content/en/docs/features/ifm-retrofitting/index.md
Outdated
Show resolved
Hide resolved
JeremyTheocharis
approved these changes
Apr 25, 2023
Because the original template from me had this structure as well. If we change that structure, we should either change all of the mand introduce redirects. Or we just leave it like this. |
|
Ok, then maybe in another PR we should change the structure, and maybe also add the archetype (already filed an issue for that #124) |
JeremyTheocharis
added a commit
that referenced
this pull request
May 5, 2023
* feat: first draft of Unified Namespace * feat: added historian * added links to features * added documentation for datasource v1 and v2 * added matteos and jeremys feedback * shortcodes * added analytics section * finished * fix: added Ferdinands feedback for limitations * Feat/xxx/ifm barcode nodered (#82) * Ifm-retrofitting * Ifm-retrofitting * Ifm-retrofitting * Ifm-retrofitting * Ifm-retrofitting * Data connectivity with Node-RED * # * # * barcodereader * barcodereader * barcodereader * feedback jeremy * feedback jeremy * feedback jeremy * fix: Corrected a few spelling mistakes change: first sentence, less repetitive add: Step by Step instruction for the barcode reader * fix: Corrected a few spelling mistakes, rephrased stuff etc. add: -Step by Step instruction for sensorconnect -Links to ifm and the retrofitting blog article -info for analog and digital sensors * add: Link to the community repo * fix: spelling * fix: spelling * included Matteos Feedback --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: David Mackel <[email protected]> * feat: finishing touches --------- Co-authored-by: krgrsebastian <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: krgrsebastian <[email protected]> Co-authored-by: David Mackel <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added retrofitting with ifm gateways, data connectivity with node-RED, and barcodereaders to the features section