Skip to content

Add OneKey Classic 1S to wallet list#4655

Open
Shoyo1024 wants to merge 1 commit intobitcoin-dot-org:masterfrom
Shoyo1024:add-onekey-classic1s
Open

Add OneKey Classic 1S to wallet list#4655
Shoyo1024 wants to merge 1 commit intobitcoin-dot-org:masterfrom
Shoyo1024:add-onekey-classic1s

Conversation

@Shoyo1024
Copy link
Copy Markdown

This PR adds the OneKey Classic 1S hardware wallet to the wallet list, incorporating all feedback from the previous review at #4439 by @crwatkins.

Per @crwatkins's recommendation, this PR covers only the OneKey Classic 1S. OneKey Pro will be submitted in a separate PR once the remaining items on that device are resolved.

Firmware reviewed: v3.18.0 (https://github.com/OneKeyHQ/firmware-classic1s/releases/tag/v3.18.0)

Issues from #4439 addressed in this PR

  • Transparency score updated to checkpasstransparencyopenspechardware (per @crwatkins's recommendation, reflecting that the THD89 secure element is closed source while a public interface specification is now available for blackbox testing).
  • Description no longer claims "fully open source" — it now accurately describes the firmware as open source.
  • "Secure Elements" lowercased to "secure elements".
  • "Open-source" rewritten without the hyphen for consistency with the rest of the wallet list.

Issues from #4439 addressed outside this PR

cc @crwatkins

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.

1 participant