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

Fix android loading state on new wallets #5082

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

jinchung
Copy link
Member

Fixes APP-762

Testing instructions in Linear

@linear
Copy link

linear bot commented Sep 28, 2023

APP-762 newWalletFlow: empty wallet state can get stuck on loading

this loading state could have been affected but the REST endpoint changes

@jinchung jinchung changed the title @jin/empty wallets loading Fix android loading state on new wallets Sep 28, 2023
@jinchung jinchung force-pushed the @jin/empty-wallets-loading branch from f672247 to cdbb53a Compare September 28, 2023 14:46
Copy link
Contributor

@dereknelson dereknelson left a comment

Choose a reason for hiding this comment

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

followed linear steps, loaded a new wallet on android and it worked x2

Copy link
Contributor

@skylarbarrera skylarbarrera left a comment

Choose a reason for hiding this comment

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

working 4 me

@@ -144,6 +147,9 @@ async function userAssetsQueryFunction({

return parsedSuccessResults;
} catch (e) {
dispatch({
Copy link
Contributor

Choose a reason for hiding this comment

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

so this is happening because we are trying to load data that does not exist?

@jinchung jinchung force-pushed the @jin/empty-wallets-loading branch from cdbb53a to e9451e1 Compare September 29, 2023 18:48
@jinchung jinchung merged commit bca9b45 into develop Sep 29, 2023
@jinchung jinchung deleted the @jin/empty-wallets-loading branch September 29, 2023 19:58
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.

3 participants