Skip to content

Battery connector spec [TC-342] #275

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

Closed
John-Page opened this issue Jun 5, 2022 · 4 comments · Fixed by #416
Closed

Battery connector spec [TC-342] #275

John-Page opened this issue Jun 5, 2022 · 4 comments · Fixed by #416
Assignees
Labels
bug Something isn't working community Bugs and fixes suggested by the community Product page

Comments

@John-Page
Copy link

It is not enough to say the battery connector is a JST type. There are hundreds of different JST connectors. All references to this should state that the connector is JST-PH which is the specific type. If you purchase batteries for this they often come with the wrong kind of JST plug.

@marqdevx
Copy link
Member

marqdevx commented Jun 8, 2022

Hey @John-Page !
Could you point the board's documentation you are talking about?
Thanks :)

@John-Page
Copy link
Author

John-Page commented Jun 8, 2022 via email

@marqdevx
Copy link
Member

marqdevx commented Jun 9, 2022

Got it, thanks for spotting it! 😄
We will take care of saying the correct ID of the connector

@per1234 per1234 added bug Something isn't working Product page community Bugs and fixes suggested by the community labels Jul 17, 2022
@jhansson-ard jhansson-ard changed the title Battery connector spec Battery connector spec [TC-342] Jul 26, 2022
@aliphys aliphys linked a pull request Jul 26, 2022 that will close this issue
1 task
@karlsoderby
Copy link
Contributor

@John-Page the MKR board product pages & tutorials have been updated with the "JST PH" specification. This can be viewed in (#416). Datasheet is not changed yet. Thank you for reporting.

@jhansson-ard jhansson-ard linked a pull request Jul 27, 2022 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working community Bugs and fixes suggested by the community Product page
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants