-
Notifications
You must be signed in to change notification settings - Fork 302
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
[LIVE-11184][Europa] Firmware update support #6775
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ 5 Ignored Deployments
|
904124e
to
9b340e8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a small refacto here, for improved readability
@@ -166,7 +166,6 @@ const EarlySecurityChecks = ({ | |||
closeFwUpdateDrawer(); | |||
restartChecksAfterUpdate(); | |||
}, | |||
deviceHasPin: deviceModelId !== DeviceModelId.stax, // early security checks are triggered only if the device is in one of the steps prior to setting a PIN code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this prop gets overridden so it's useless here
β Checklist
npx changeset
was attached.π Description
Some fixes necessary to fully support firmware updates of Europa via LLM & LLD:
device-core
: the minimum supported version forisBootloaderVersionSupported
andisDeviceLocalizationSupported
were incorrect for Europa, which led to an incorrect parsing of thegetVersion
response:LLD
: the "restore" step of the firmware update was only added for devices supporting device localisation. It should also take into account support of custom lock screen. Given the issue listed previously, Europa was considered as not supporting localisation, therefore highlighting this issue because the restore step was missing, and the update flow broken.β Context
π§ Checklist for the PR Reviewers