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

Allow to reload extension from within error modal #1860

Merged
merged 3 commits into from
Apr 2, 2024
Merged

Conversation

buberdds
Copy link
Contributor

@buberdds buberdds commented Mar 28, 2024

Fixes #1853

Don is fine with ext layout variant.

Base automatically changed from mz/fatalErr to master March 29, 2024 07:50
Copy link

github-actions bot commented Mar 29, 2024

Deployed to Cloudflare Pages

Latest commit: 554969f2560b6e49cebd2b67a2bdeb03d06d94de
Status:✅ Deploy successful!
Preview URL: https://22b62f45.oasis-wallet.pages.dev

Copy link

codecov bot commented Mar 29, 2024

Codecov Report

Attention: Patch coverage is 87.50000% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 81.30%. Comparing base (9fd6c69) to head (554969f).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1860   +/-   ##
=======================================
  Coverage   81.29%   81.30%           
=======================================
  Files         192      193    +1     
  Lines        5084     5097   +13     
  Branches      936      941    +5     
=======================================
+ Hits         4133     4144   +11     
- Misses        951      953    +2     
Flag Coverage Δ
cypress 45.94% <18.18%> (-0.10%) ⬇️
jest 77.18% <87.50%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/app/components/Button/index.tsx 100.00% <100.00%> (ø)
src/app/components/FatalErrorHandler/index.tsx 88.00% <80.00%> (-2.70%) ⬇️

@buberdds buberdds force-pushed the mz/reloadExt branch 3 times, most recently from c8edcd2 to f85c2d6 Compare March 29, 2024 10:37
@buberdds buberdds marked this pull request as ready for review March 29, 2024 10:43
src/app/components/Button/index.tsx Outdated Show resolved Hide resolved
src/app/components/Button/index.tsx Outdated Show resolved Hide resolved
/>
</Box>
{runtimeIs === 'extension' && (
<Box align={isMobile ? 'stretch' : 'end'} margin={{ bottom: isMobile ? 'xlarge' : 'large' }}>
Copy link
Member

Choose a reason for hiding this comment

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

preview
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will ask Don tomorrow if we want to move this button up

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Screenshot from 2024-04-02 11-28-37

@buberdds buberdds force-pushed the mz/reloadExt branch 3 times, most recently from 06b1468 to 03d6ebd Compare April 1, 2024 18:07
@buberdds buberdds merged commit ddb89e1 into master Apr 2, 2024
14 of 15 checks passed
@buberdds buberdds deleted the mz/reloadExt branch April 2, 2024 09: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.

Extension bug: if user has no connection when starting up the browser then popup displays RPC error forever
2 participants