Skip to content

Conversation

@DavidsonMSD
Copy link
Contributor

  • Set up the initial project structure.
    • Added main project files.
    • Added assets.
    • Added PLC IDE code (including both Ladder and Sketch).

What This PR Changes

  • This PR adds the initial version of the conveyor and sorting application note.

Contribution Guidelines

- Set up the initial project structure.
- Added main project files.
- Added assets.
- Added code (both Ladder and Sketch).
@DavidsonMSD DavidsonMSD requested a review from mcmchris September 2, 2024 13:17
@CLAassistant
Copy link

CLAassistant commented Sep 2, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@mcmchris mcmchris left a comment

Choose a reason for hiding this comment

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

@DavidsonMSD great application note, I like the way you adapt to suggestions and templates.
I left some comments and suggestions mainly focused on format, titles and minor stuff. Feel free to ask for more details about them.
cc. @jcarolinares

@jcarolinares jcarolinares changed the title Initial commit: Conveyor and sorting AppNote [PC-1929] Initial commit: Conveyor and sorting AppNote Sep 6, 2024
@github-actions
Copy link

github-actions bot commented Sep 6, 2024

Preview Deployment

🚀 Preview this PR: https://66fd4edff50664fc061b87f7--docs-content.netlify.app
📍 Commit SHA: 778e980

Added missing images
Replaced SVG images with PNG format
Modified some text sections
Added I/O mapping section
Addressed review comments
Added start and stop buttons
@DavidsonMSD DavidsonMSD requested a review from mcmchris October 2, 2024 02:33
DavidsonMSD and others added 3 commits October 1, 2024 23:43
Correcting the error detected in Validate Content.
@mcmchris mcmchris removed the preview label Oct 2, 2024
Copy link
Collaborator

@mcmchris mcmchris left a comment

Choose a reason for hiding this comment

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

This is ready to publish! Thanks @DavidsonMSD for doing a great job with this Application Note!

@jcarolinares
Copy link
Contributor

Merging

@jcarolinares jcarolinares merged commit a56f58a into main Oct 2, 2024
4 checks passed
@jcarolinares jcarolinares deleted the sync/davidonmsd/opta/conveyor-sorting-app-note branch October 2, 2024 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants