Skip to content
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

Merged
merged 10 commits into from
Feb 2, 2024

Conversation

InfraredAces
Copy link
Contributor

No description provided.

Copy link
Contributor

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?

Copy link
Contributor

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?

Copy link
Contributor Author

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.

Copy link
Contributor

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.  

docs/installation.mdx Outdated Show resolved Hide resolved
Copy link
Contributor

@TheTrainGoes TheTrainGoes left a 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.

@TheTrainGoes
Copy link
Contributor

@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?

@TheTrainGoes TheTrainGoes merged commit 4f15204 into OpenStickCommunity:main Feb 2, 2024
1 check passed
@InfraredAces InfraredAces deleted the 0.7.7-update branch February 2, 2024 18:48
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