Skip to content

Customer feedback on battery replacement #170

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 1 commit into from
Jul 17, 2025

Conversation

thomas-zimmerman
Copy link
Contributor

Add some additional advise on removing the battery and how to connect the battery again.

@thomas-zimmerman thomas-zimmerman requested review from ahoneybun and a team March 28, 2023 15:58
@leviport
Copy link
Member

I believe these sections are pretty much identical between lemp9, lemp10, and lemp11. Should we make similar changes in lemp11 and lemp9 as well?

@jacobgkau jacobgkau force-pushed the thomas-zimmerman-lemp10-battery-updates branch from 8fcbe7c to 33b3f40 Compare April 24, 2023 20:53
@jacobgkau
Copy link
Member

@thomas-zimmerman What is the actual recommended method you want the instructions to say? We have:

  • (existing) Wedge a plastic tool between the battery/tape to separate them.
  • (existing) Pull the tabs up to separate the tape and battery from the chassis.
  • (new) "Pull the tabs to remove most of the adhesive" is what you originally added here-- I assume you mean pulling it outward, rather than up away from the chassis as was already suggested?

I think this is too many alternatives-- we're basically not giving people instructions if we just list every possible way to go about it.

@thomas-zimmerman
Copy link
Contributor Author

@thomas-zimmerman What is the actual recommended method you want the instructions to say? We have:

  • (existing) Wedge a plastic tool between the battery/tape to separate them.
  • (existing) Pull the tabs up to separate the tape and battery from the chassis.
  • (new) "Pull the tabs to remove most of the adhesive" is what you originally added here-- I assume you mean pulling it outward, rather than up away from the chassis as was already suggested?

I think this is too many alternatives-- we're basically not giving people instructions if we just list every possible way to go about it.

They may need to use all of these alternate ways to remove the adhesive. The pull tabs can break and leave a decent amount of adhesive on the bottom of the battery, and you will be using a plastic tool to try and remove the excess.

Good catch, I do mean that they will need to pull outward from the battery, not away from the chassis.

@jacobgkau
Copy link
Member

jacobgkau commented Apr 24, 2023

Ok, I'm good with leaving a form of all three in, but two of them (as currently worded) are mutually exclusive. Before the tabs break, you have to pull them out or up, you can't really do both. Unless we just want to say "upwards and outwards" for both at once (I'm not sure if that would lessen the effectiveness of the outward pull.)

@jacobgkau jacobgkau force-pushed the thomas-zimmerman-lemp10-battery-updates branch from f55aea7 to 431c85e Compare May 25, 2023 22:39
@jacobgkau jacobgkau force-pushed the thomas-zimmerman-lemp10-battery-updates branch 2 times, most recently from 4314ee1 to 2dc2929 Compare July 16, 2025 18:56
@jacobgkau jacobgkau removed the request for review from ahoneybun July 16, 2025 18:56
@jacobgkau
Copy link
Member

@thomas-zimmerman I've fixed up the wording, attempting to parse what your initial wording meant regarding the battery connector. Do you think this version is okay?

Copy link
Contributor Author

@thomas-zimmerman thomas-zimmerman left a comment

Choose a reason for hiding this comment

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

Yes, I do think that is an improvement. No odd formatting when looking at the live view in mdbook. I'd approve the PR but not able as the author.

@jacobgkau jacobgkau force-pushed the thomas-zimmerman-lemp10-battery-updates branch from 2dc2929 to ef8b477 Compare July 17, 2025 21:07
@jacobgkau jacobgkau requested a review from leviport July 17, 2025 21:09
leviport
leviport previously approved these changes Jul 17, 2025
@jacobgkau
Copy link
Member

@thomas-zimmerman FYI, this has changed somewhat significantly again since your previous acknowledgement. I'm assuming this is still fine with you as well?

@jacobgkau jacobgkau force-pushed the thomas-zimmerman-lemp10-battery-updates branch from ef8b477 to 579a136 Compare July 17, 2025 21:14
@jacobgkau
Copy link
Member

Just had to correct the number of the final step since I'd added one.

@thomas-zimmerman
Copy link
Contributor Author

I'm good with the additional changes!

@jacobgkau jacobgkau merged commit 895ea0c into master Jul 17, 2025
1 check passed
@jacobgkau jacobgkau deleted the thomas-zimmerman-lemp10-battery-updates branch July 17, 2025 21:54
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.

3 participants