Skip to content

[PC-1968] Nicla Vision: Datasheet Topology Section Update #2204

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

Merged

Conversation

ArduinoBot
Copy link
Collaborator

@ArduinoBot ArduinoBot commented Oct 1, 2024

What This PR Changes

Updates the Nicla Vision datasheet with corrections required for the Topology section.

Contribution Guidelines

@ArduinoBot ArduinoBot added documentation Improvements or additions to documentation arduino Bugs and fixes added by the Arduino Team labels Oct 1, 2024
@TaddyHC TaddyHC added the preview label Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

Preview Deployment

🚀 Preview this PR: https://66fc8cb305f377fdacf5cdb6--docs-content.netlify.app
📍 Commit SHA: 35cd47b

Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

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

Requested changes:

  • Make both images of the diagrams smaller so each table and images fix on the same page

  • I’m having problems finding J1 and J2, are we sure they are on the diagram?

Copy link
Contributor

@jcarolinares jcarolinares left a comment

Choose a reason for hiding this comment

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

LGMT merging

@jcarolinares jcarolinares merged commit d501de3 into main Oct 3, 2024
4 checks passed
@jcarolinares jcarolinares deleted the sync/taddy/nicla-vision-datasheet-topology-update branch October 3, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arduino Bugs and fixes added by the Arduino Team documentation Improvements or additions to documentation preview
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants