search

lukso-network / universalprofile-test-dapp

Testing dApps for ERC725 and Universal Profiles.
https://up-test-dapp.lukso.tech/
17 stars 14 forks source link

fix: Adds operatorSend, operatorBurn, approve, safeTransferFrom #107

Closed andyogaga closed 1 year ago

andyogaga commented 1 year ago

https://app.clickup.com/t/866abevnp

frozeman commented 1 year ago

Task linked: CU-866abevnp Adds generic tx to test dApp

cloudflare-workers-and-pages[bot] commented 1 year ago

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4eb69dc
Status: ✅  Deploy successful!
Preview URL: https://374cf7ab.universalprofile-test-dapp.pages.dev
Branch Preview URL: https://fix-adds-generic-tx-cu-866ab.universalprofile-test-dapp.pages.dev

View logs

github-actions[bot] commented 1 year ago

PR Preview Action v1.2.0 :---: Preview removed because the pull request was closed. 2023-05-25 08:22 UTC

andyogaga commented 1 year ago

lgtm, btw we still have 'numberToHex' is defined but never used around, can you fix it?

Sorry, merged it before seeing this comment. Fixing it in my next PR