VAnchor Substrate Implementation (Missing Withdraw/Transfer Flows)#1139
Conversation
AtelyPham
changed the title
| new Worker( | ||
| new URL( | ||
| '@webb-tools/react-environment/arkworks-proving-manager.worker' | ||
| '@webb-tools/react-environment/arkworks-proving-manager.worker', |
There was a problem hiding this comment.
We should also consider having a way to test out using the same flow for Circom proving (for EVM) on the Substrate side as well. We do have a test showing that those proofs are valid. Possible new task.
drewstone
left a comment
drewstone
left a comment
There was a problem hiding this comment.
LG, big change here. Granted we want to use Circom proving for all sides of the bridge, that is my main feedback here. We should create a new task to use the same proving pipeline for any side, and work to deprecate the WASM/Web Worker pattern in favor for what's already on the EVM sides.
|
@AtelyPham please resolve conflict |
|
✅ Deploy Preview for stats-dapp development is ready! Thanks for the contribution @AtelyPham
To edit notification comments on pull requests, go to your Netlify site settings. |
|
✅ Deploy Preview for bridge-dapp development is ready! Thanks for the contribution @AtelyPham
To edit notification comments on pull requests, go to your Netlify site settings. |
|
✅ Deploy Preview for webb faucet is ready! Thanks for the contribution @AtelyPham
To edit notification comments on pull requests, go to your Netlify site settings. |
|
✅ Deploy Preview for stats-dapp development is ready! Thanks for the contribution @AtelyPham
To edit notification comments on pull requests, go to your Netlify site settings. |
|
✅ Deploy Preview for bridge-dapp development is ready! Thanks for the contribution @AtelyPham
To edit notification comments on pull requests, go to your Netlify site settings. |
|
✅ Deploy Preview for webb faucet is ready! Thanks for the contribution @AtelyPham
To edit notification comments on pull requests, go to your Netlify site settings. |
5 participants
Summary of changes
getNextIndex,getLeafIndex, ...).Scripts added
yarn nx run bridge-dapp:run-local-substrate: Run and setup the environment for localprotocol-substrate(read info here before running the script).How to setup and test Substrate <> Substrate deposit flow
protocol-substraterepo in the same level as thewebb-dapprepo in your machine.yarn nx run bridge-dapp:run-local-substrateon your terminal.✅ Protocol Substrate ready to use!!!message shows on the terminal.developmentoptionprotocol-substrate.Proposed area of change
apps/bridge-dappapps/stats-dappapps/webbsiteapps/faucetapps/tangle-websitelibs/webb-ui-componentsReference issue to close (if applicable)
NaNScreen Recording
CleanShot.2023-04-29.at.03.11.01.mp4