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: Add spinner to email sign in #15

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Conversation

jophish
Copy link

@jophish jophish commented Oct 15, 2024

See slack thread here

@jophish jophish merged commit 5332979 into main Oct 15, 2024
8 of 10 checks passed
@jophish jophish deleted the jophish/fix-mnemonic-race branch October 15, 2024 21:17
github-merge-queue bot pushed a commit to valora-inc/wallet that referenced this pull request Oct 21, 2024
…mes (#6177)

### Description

Applying the
[fix](mobilestack-xyz#15) that
we made in the shefi project.

### Test plan

n/a

### Related issues

n/a

### Backwards compatibility

Y

### Network scalability

If a new NetworkId and/or Network are added in the future, the changes
in this PR will:

- [ ] Continue to work without code changes, OR trigger a compilation
error (guaranteeing we find it when a new network is added)

Co-authored-by: Joe Bergeron <jbergero@alum.mit.edu>
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.

1 participant