Skip to content
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

🔥 Hotfix 2024-07-02 #7241

Merged
merged 13 commits into from
Jul 3, 2024
Merged

🔥 Hotfix 2024-07-02 #7241

merged 13 commits into from
Jul 3, 2024

Conversation

valpinkman
Copy link
Member

Next release is on the way 🚀!

This pull request aims to fix a bug in production for a Ledger Live product.

Checklist

To be ready to be merged, we must comply with all theses checks:

  • changelogs are included with description about the bug(s)
  • all CI checks have passed
  • reviewed by QA
  • hotix is reviewed by the Delivery Team

⚠️ If this hotfix targets Ledger Live Desktop

  • RELEASE_NOTES.md has been updated

Once all these checks are validated, the pull request can be merged using the correct workflow.

Copy link

vercel bot commented Jul 2, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web-tools ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 3, 2024 9:32am
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
ledger-live-docs ⬜️ Ignored (Inspect) Visit Preview Jul 3, 2024 9:32am
ledger-live-github-bot ⬜️ Ignored (Inspect) Visit Preview Jul 3, 2024 9:32am
native-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Jul 3, 2024 9:32am
react-ui-storybook ⬜️ Ignored (Inspect) Visit Preview Jul 3, 2024 9:32am

@valpinkman valpinkman requested a review from a team as a code owner July 2, 2024 09:54
@live-github-bot live-github-bot bot added the mobile Has changes in LLM label Jul 2, 2024
smartine-ledger
smartine-ledger previously approved these changes Jul 2, 2024
mbertin-ledger
mbertin-ledger previously approved these changes Jul 2, 2024
@live-github-bot live-github-bot bot requested a review from a team as a code owner July 2, 2024 10:02
@valpinkman valpinkman requested a review from a team as a code owner July 2, 2024 12:49
@live-github-bot live-github-bot bot added common Has changes in live-common ledgerjs Has changes in the ledgerjs open source libs labels Jul 2, 2024
KVNLS
KVNLS previously approved these changes Jul 3, 2024
Copy link
Member

@KVNLS KVNLS left a comment

Choose a reason for hiding this comment

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

LGTM! Just a comment to improve readability for a test

Comment on lines +102 to 106
(connectionType === "bluetooth" &&
Platform.OS === "android" &&
!!lastSeenDeviceModelInfo &&
semver.lt(lastSeenDeviceModelInfo?.deviceInfo.version, "2.4.0")),
};
Copy link
Contributor

Choose a reason for hiding this comment

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

this is missing a condition on the device model no?

Copy link
Member Author

@valpinkman valpinkman Jul 3, 2024

Choose a reason for hiding this comment

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

I would say isOldUxSupportedButNotWired already takes care of that but you might be right.....

@valpinkman valpinkman merged commit 1328245 into main Jul 3, 2024
15 of 16 checks passed
@valpinkman valpinkman deleted the hotfix branch July 3, 2024 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli common Has changes in live-common desktop Has changes in LLD ledgerjs Has changes in the ledgerjs open source libs mobile Has changes in LLM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants