Skip to content

[TC-412] [PC-1057] Revise Nicla Sense ME battery image #729

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
merged 6 commits into from
Dec 20, 2022

Conversation

aliphys
Copy link
Contributor

@aliphys aliphys commented Dec 13, 2022

What This PR Changes

  • Update the Nicla Sense ME battery image in the datasheet so that the black wire is at the top of the board
  • image

Contribution Guidelines

@aliphys aliphys added Datasheet fix/update A small fix or update labels Dec 13, 2022
@aliphys aliphys requested a review from jcarolinares December 13, 2022 09:53
@gatsby-cloud
Copy link

gatsby-cloud bot commented Dec 13, 2022

✅ docs-content deploy preview ready

@aliphys
Copy link
Contributor Author

aliphys commented Dec 15, 2022

@jcarolinares Can we move ahead and merge this PR? :)

@canchebagur canchebagur self-requested a review December 15, 2022 14:01
@canchebagur
Copy link
Contributor

@aliphys I was checking this PR today in the backlog/PR cleaning. It looks good to me, but I would also add a note somewhere that explicitly tells the orientation of the battery's cable in the board besides the change in the image. Something like "Notice the orientation of the battery's cable in the board's connector."

image

@jcarolinares
Copy link
Contributor

@aliphys review done

you have my go to merge, thank you

@aliphys aliphys merged commit 630997f into main Dec 20, 2022
@aliphys aliphys deleted the aliphys/niclaNewBatImage branch December 20, 2022 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Datasheet fix/update A small fix or update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants