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

Disable HW on firefox #907

Merged
merged 6 commits into from
Aug 31, 2023
Merged

Disable HW on firefox #907

merged 6 commits into from
Aug 31, 2023

Conversation

brunobar79
Copy link
Member

Fixes BX-####
Figma link (if any):

What changed (plus any additional context for devs)

Disable HW on firefox including e2e. Showing an alert, see below.

Screen recordings / screenshots

Screen Shot 2023-08-30 at 5 58 35 PM

What to test

Final checklist

  • I have tested my changes in a LavaMoat bundle (yarn build).
  • I have tested my changes in Chrome & Brave.
  • If your changes are visual, did you check both the light and dark themes?

@brunobar79 brunobar79 requested a review from a team August 30, 2023 22:01
@github-actions
Copy link

Here's the packed extension for this build:
node_modules.tar.gz
rainbowbx-3a7ba46d86ffccdcaaf45506f5a6ece3d9c7f43f.zip

@github-actions
Copy link

Here's the packed extension for this build:
rainbowbx-1a50a1cdd669d61c9e4af34fab1cd8537d5e3d47.zip

@@ -1031,7 +1031,8 @@
"alert": {
"ok": "Ok",
"wallet_watching_mode": "This wallet is currently in 'Watching' mode",
"coming_soon": "Coming Soon!"
"coming_soon": "Coming Soon!",
"no_hw_ff": "Harware wallets are not supported on Firefox. Try a different Browser like Chrome or Brave."
Copy link
Member

Choose a reason for hiding this comment

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

"Harware" -> "Hardware"

"Browser" -> "browser"

Copy link
Member Author

Choose a reason for hiding this comment

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

Hard fail

@github-actions
Copy link

Here's the packed extension for this build:
node_modules.tar.gz
rainbowbx-255ed7020c23fe00393dc81b1621b4b083b03e75.zip

@github-actions
Copy link

Here's the packed extension for this build:
rainbowbx-255ed7020c23fe00393dc81b1621b4b083b03e75.zip

@brunobar79 brunobar79 merged commit 292e3bd into master Aug 31, 2023
14 checks passed
@brunobar79 brunobar79 deleted the @bruno/ledger-ff branch August 31, 2023 22:59
@github-actions
Copy link

Here's the packed extension for this build:
node_modules.tar.gz
rainbowbx-292e3bd8f6798841ee0ff5f3089882e035da821b.zip

greg-schrammel pushed a commit that referenced this pull request Sep 4, 2023
greg-schrammel pushed a commit that referenced this pull request Sep 6, 2023
@jcmonte
Copy link

jcmonte commented Nov 7, 2023

What was the reason for disabling firefox hardware wallet support?

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.

None yet

4 participants