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

Replacing UAL with Wharf #646

Draft
wants to merge 24 commits into
base: master
Choose a base branch
from
Draft

Conversation

aaroncox
Copy link
Contributor

@aaroncox aaroncox commented Mar 7, 2023

Description

There's still a lot to be done and this is a draft PR not meant to be merged in (at least not yet). It's my exploration integrating the Wharf Session Kit into OBE and replacing all of the UAL code/hacks OBE currently depends on.

I have a lot of comments about how OBE could be simplified and how the user interaction flow could be modified - but need to find the time to write all that up. I'm putting this up as a draft pull request just for visibility and discussion 👍

@netlify
Copy link

netlify bot commented Mar 7, 2023

Deploy Preview for obe-testnet ready!

Name Link
🔨 Latest commit c9667b5
🔍 Latest deploy log https://app.netlify.com/sites/obe-testnet/deploys/649cbbc9db919800089cfab7
😎 Deploy Preview https://deploy-preview-646--obe-testnet.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Mar 7, 2023

Deploy Preview for open-block-explorer ready!

Name Link
🔨 Latest commit c9667b5
🔍 Latest deploy log https://app.netlify.com/sites/open-block-explorer/deploys/649cbbc91add6200087163a5
😎 Deploy Preview https://deploy-preview-646--open-block-explorer.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@aaroncox
Copy link
Contributor Author

aaroncox commented Mar 7, 2023

Also just to note I haven't implemented any "real" wallet plugins, so this is only really working well with the Jungle 4 configuration. As soon as the Anchor wallet plugin is ready (we're still fixing bugs), I'll get that added in and it should work with all networks.

@aaroncox
Copy link
Contributor Author

aaroncox commented Mar 7, 2023

The MockWalletPlugin is set now to use whatever network you're on, and simulate a login to the eosio@active account for testing purposes. Every transaction you try to perform is going to fail (for obvious reasons), but it'll at least allow interaction with the login/logout/restore/transact flows.

@aaroncox
Copy link
Contributor Author

aaroncox commented Mar 9, 2023

I've added the Anchor and cleos wallet plugins now, I think it's mostly functional. Still cleanup to do on our end as well as in the OBE to remove all the other cleos presentation stuff.

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