Skip to content

Made the fix to create the atomic transaction.#68

Open
Jamest145 wants to merge 1 commit intoalgorand-coding-challenges:mainfrom
Jamest145:main
Open

Made the fix to create the atomic transaction.#68
Jamest145 wants to merge 1 commit intoalgorand-coding-challenges:mainfrom
Jamest145:main

Conversation

@Jamest145
Copy link

Fix the Bug Submission Pull Request

What was the bug?

The bug was that the two transactions were not being signed, and that TransactionWithSigner was not being passed to the atc.addTransaction method.

How did you fix the bug?

I used the getTransactionWithSigner method, evoked it two times, once for each transaction. Signed it with the sender account, and then added the transaction to the atomic transaction composer.

Console Screenshot:

image

@iskysun96 iskysun96 added the Approved ✅ Successfully completed the challenge! label Apr 14, 2024
@iskysun96
Copy link
Contributor

🎉 Congrats on solving the 4th Algorand Coding Challenge!

I have labeled your PR approved, so you can now claim your certificate.

You can claim your certificate of completion NFT here:
https://goplausible.xyz/claim/8x3aqnmh7p2wf25dvmxz7ksvjs3gr02d6648tf31ny90wwtams53jvssjwcj2bgfs7qr425958h3abw83a33t3tdbqb0sjtntmta82r

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

Labels

Approved ✅ Successfully completed the challenge!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants