pyteal icon indicating copy to clipboard operation
pyteal copied to clipboard

Uday30

Open uday7981 opened this issue 2 years ago • 2 comments

Hey, I have just updated the both examples:

In the first code snippet, the Algorand Node API was used to compile the smart contract TEAL code and generate a logic signature using the goal command-line tool. In the updated code, the Python SDK is used to compile the TEAL code and generate the logic signature using the algod_client.compile method. This eliminates the need to use the goal command-line tool and simplifies the code.

In the second code snippet, the logic signature was attached to a payment transaction and then sent to the Algorand network using the algod command-line tool. In the updated code, the Python SDK is used to create and send the transaction using the algod_client.send_transactions method. This simplifies the code and provides a more consistent way to interact with the Algorand network.

uday7981 avatar Mar 01 '23 11:03 uday7981

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: uday7981
:x: udaykirannnnnn
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Mar 01 '23 11:03 CLAassistant

Hi, thanks for updating examples, and there are a few things to notice before you would like to merge in:

  1. Please consider using formatter to format your code, CI process failed for not formatting your code properly. A convenient way to setup environment and everything you need is going through https://github.com/algorand/pyteal#development-setup
  2. Please consider update your SDK version. From your code example it seems you are still on py-sdk v1, please use updated version v2.
  3. Is this PR duplicating with https://github.com/algorand/pyteal/pull/640? If so please remain only one of them.
  4. There are 2 committers (assuming both are your accounts) on your branch, then please consider let both sign CLA.

Let me know if anything I can help.

ahangsu avatar Mar 01 '23 14:03 ahangsu