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

Add default error page #5657

Merged
merged 4 commits into from
Apr 23, 2024
Merged

Add default error page #5657

merged 4 commits into from
Apr 23, 2024

Conversation

brunobar79
Copy link
Member

Fixes APP-####

What changed (plus any additional context for devs)

Screen recordings / screenshots

Screen.Recording.2024-04-22.at.5.25.00.PM.mov

What to test

import lang from 'i18n-js';
import React from 'react';
import { Centered } from '../layout';
import { Text } from '../text';
Copy link
Contributor

Choose a reason for hiding this comment

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

you're playing with fire

Copy link
Member Author

Choose a reason for hiding this comment

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

I heard you're great at making new screens

Copy link
Member

@benisgold benisgold left a comment

Choose a reason for hiding this comment

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

it's beautiful. my 1 nit pick is that when this screen is shown is it ever actually possible that reloading the page as suggested will fix the problem?

@brunobar79
Copy link
Member Author

it's beautiful. my 1 nit pick is that when this screen is shown is it ever actually possible that reloading the page as suggested will fix the problem?

That's a good point. For most errors it won't be recoverable so the current copy isn't ideal. Do you have any suggestion?

brunobar79 and others added 3 commits April 23, 2024 13:52
Co-authored-by: Matthew Wall <matthew.wallt@gmail.com>
Co-authored-by: Matthew Wall <matthew.wallt@gmail.com>
Co-authored-by: Matthew Wall <matthew.wallt@gmail.com>
@brunobar79 brunobar79 merged commit 22c658b into develop Apr 23, 2024
4 of 5 checks passed
@brunobar79 brunobar79 deleted the @bruno/default-error-page branch April 23, 2024 17:54
christianbaroni added a commit that referenced this pull request Apr 30, 2024
BrodyHughes added a commit that referenced this pull request May 7, 2024
…eplink-add

* 'develop' of github.com:rainbow-me/rainbow:
  simplify logic for fee display (#5660)
  zustand: use smol subscriptions (#5661)
  useSyncSharedValue (#5662)
  handle wc links in browser (#5652)
  Add default error page (#5657)
  update firebase (#5636)
  browser: fix android card images (#5651)
  Add backend provided slippage (#5649)
  Add rainbow fee to review panel (#5650)
  review prompt changes (#5656)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants