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.

Sign up for GitHub

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

Jump to bottom

feat: web wallet composite #236

Closed
wants to merge 31 commits into from
Closed

feat: web wallet composite #236

wants to merge 31 commits into from

Conversation

leonardocbsr
Copy link
Collaborator

@leonardocbsr leonardocbsr commented Sep 5, 2024 •
edited
Loading

Context

By using the hooks in the React package, we were able to create a WebWallet and ship it in the @fuels/react package.
This can be enabled by using showWebWallet in the FuelProvider.
The default is disabled.

This PR

Screenshots

In the comments below.

@leonardocbsr leonardocbsr self-assigned this Sep 5, 2024
@vercel Vercel
Copy link

vercel bot commented Sep 5, 2024

@leocourbassier is attempting to deploy a commit to the Fuel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@leonardocbsr leonardocbsr marked this pull request as ready for review September 6, 2024 10:27
@leonardocbsr leonardocbsr enabled auto-merge (squash) September 6, 2024 10:28
@vercel Vercel
Copy link

vercel bot commented Sep 6, 2024 •
edited
Loading

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

Name Status Preview Comments Updated (UTC)
fuel-connectors ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 23, 2024 6:01am

@leonardocbsr
Copy link
Collaborator Author

Depends on FuelLabs/fuel-explorer#569

@leonardocbsr leonardocbsr added the blocked Blocked by another issue label Sep 20, 2024
@leonardocbsr leonardocbsr removed the blocked Blocked by another issue label Sep 21, 2024
@leonardocbsr
Copy link
Collaborator Author

Light Theme

@leonardocbsr
Copy link
Collaborator Author

Dark Theme

@leonardocbsr leonardocbsr linked an issue Sep 21, 2024 that may be closed by this pull request
Web Wallet view of connected account #207
Open
LuizAsFight
LuizAsFight reviewed Sep 21, 2024
View reviewed changes
Copy link
Contributor

@LuizAsFight LuizAsFight left a comment

Choose a reason for hiding this comment

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

check all packages.json to see if we're unnecessarily downgrading any deps

examples/react-app/package.json Show resolved Hide resolved
examples/react-app/package.json Show resolved Hide resolved
examples/react-next/package.json Show resolved Hide resolved
@leonardocbsr
Copy link
Collaborator Author

This feature will continue on #281

@leonardocbsr leonardocbsr closed this by deleting the head repository Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luizstacio luizstacio Awaiting requested review from luizstacio luizstacio is a code owner

@pedronauck pedronauck Awaiting requested review from pedronauck pedronauck is a code owner

@helciofranco helciofranco Awaiting requested review from helciofranco helciofranco is a code owner

@arthurgeron arthurgeron Awaiting requested review from arthurgeron arthurgeron is a code owner

@rodrigobranas rodrigobranas Awaiting requested review from rodrigobranas rodrigobranas is a code owner

@LuizAsFight LuizAsFight Awaiting requested review from LuizAsFight LuizAsFight is a code owner

Assignees

@leonardocbsr leonardocbsr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Web Wallet view of connected account
4 participants
@leonardocbsr @luizstacio @LuizAsFight @leocourbassier