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

Fix result of signTransaction #8

Closed

fabian-emilius wants to merge 1 commit into greymass:master from fabian-emilius:master

fabian-emilius commented Jun 28, 2020

SignTransactionResponse does not contain a transaction id with ual anchor, which may be needed by dapps


          Fix transaction return

0820f4d

Member

aaroncox commented Feb 13, 2021

Pretty sure we talked about this and I just never closed this, but this was implemented.

ual-anchor/src/AnchorUser.ts

Lines 42 to 45 in ff8c9bc

    
           return this.returnEosjsTransaction(wasBroadcast, { 
        
             transaction_id: completedTransaction.payload.tx, 
        
             ...completedTransaction 
        
           })

Closing this out just to clean up the issues 👍

aaroncox closed this

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

Labels

None yet