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

chore: Update Leap Capsule Social Login #14

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jlm0
Copy link

@jlm0 jlm0 commented Jul 11, 2024

The PR updates the Leap social login dependency. The latest version requires transpilation in Next.js configs. Additional CSS styles were added to the parent tag for the custom capsule modal.
We are still investigating an issue where, when using an existing Capsule account, the EVM and Cosmos Kit wallet connectors are not updating despite the Capsule connection being active.

Copy link

vercel bot commented Jul 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
celestine-sloths ❌ Failed (Inspect) Jul 11, 2024 3:40pm

@jlm0
Copy link
Author

jlm0 commented Jul 11, 2024

@karan-leapwallet I closed the other open PR and consolidated to a single. Note that the modal not showing on top of the wallet connector also required !z-[99999] to work properly.
Still investigating edge case for existing capsule accounts coming from evm to cosmos and the cosmos kit not updating state despite the connection being active.

@karan-leapwallet
Copy link
Collaborator

@karan-leapwallet I closed the other open PR and consolidated to a single. Note that the modal not showing on top of the wallet connector also required !z-[99999] to work properly. Still investigating edge case for existing capsule accounts coming from evm to cosmos and the cosmos kit not updating state despite the connection being active.

Even signout -> signin again modal not appearing case is fixed?

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.

2 participants