Skip to content

Conversation

@tomiiide
Copy link
Contributor

@tomiiide tomiiide commented Jul 24, 2025

Which Jira task is linked to this PR?

https://lifi.atlassian.net/browse/LF-14696

Why was it implemented this way?

It was implemented the standard way we implement these kinds of wallets.

Visual showcase (Screenshots or Videos)

Screenshot 2025-07-29 at 11 31 50

Checklist before requesting a review

  • I have performed a self-review of my code.
  • This pull request is focused and addresses a single problem.
  • If this PR modifies the Widget API, I have updated the documentation (or submitted a change request on GitBook).

@tomiiide tomiiide marked this pull request as ready for review July 29, 2025 09:59
@github-actions
Copy link

Hey! This is your new endopint: https://d787521b.widget-lf14696add.pages.dev

@tomiiide tomiiide requested review from chybisov and effie-ms and removed request for chybisov July 29, 2025 10:31
Copy link
Member

@chybisov chybisov left a comment

Choose a reason for hiding this comment

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

Can you please check if we still need Coinbase? 👀

@tomiiide
Copy link
Contributor Author

tomiiide commented Jul 29, 2025

Can you please check if we still need Coinbase? 👀

@chybisov Yeah we do for one reason, it is the only way to connect with the Coinbase app wallet on mobile.

Screenshot 2025-07-29 at 12 38 50

However, the Coinbase wallet automatically connects to an existing base account connection, and also give you an option to login in to one during the auth flow. (which makes things a bit confusing)

Also, we should rename the Coinbase wallet to the Base App, and update the logo.

@chybisov
Copy link
Member

@tomiiide, ah I see, they will probably merge them at some point, but for now, let's keep both. Thank you!

@chybisov
Copy link
Member

Just got an update 😁

image

@chybisov chybisov merged commit eff8baa into main Jul 31, 2025
1 check passed
@chybisov chybisov deleted the LF-14696-add-base-account-support branch July 31, 2025 13:51
@github-actions
Copy link

Hey! This is your new endopint: https://868164e0.widget-lf14696add.pages.dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants