-
Notifications
You must be signed in to change notification settings - Fork 13
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
Update and corrections for 0.7.7 #15
Update and corrections for 0.7.7 #15
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Conversation note - do we want to have add ons shown here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a need to re-label the not-in-use GPIO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The labels are meant to just indicate that there is a place to label the button as it's a template. For someone adding a board they would toggle visibility on the components that are not needed. For example, if someone were placing the flag on the right of a board, they would toggle the left connector and the right label, leaving only |Left Label|GPIO BOX|Right Connector|
Primarily to have a consistent style for labeling boards.
docs/controller-build/wiring.mdx
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vendors will often take these pin outs and modify them after flashing in order to get them working on their particular devices. So be careful when performing a reset on your controller.
If the board is ever reset through the Web Configurator, your controller may no longer function as intended and these may be the assigned pin mapping of the board.
:::```
This section needs to be changed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some notes and some conversation points - feel free to reach out on Discord.
Sorry I did not see this PR.
@InfraredAces I am going to approve and merge this and we can make further changes after. If possible, moving forward, can we have seperate PRs for individual things to make it easier to manage / work on? |
No description provided.