refactor(examples): modularize token_transfer.py for clarity and reuse (#773) #782
Conversation
Signed-off-by: Raja Rathour <imraja729@gmail.com>
…ledger#773) Signed-off-by: Raja Rathour <imraja729@gmail.com>
Signed-off-by: Raja Rathour <193507075+Raja-89@users.noreply.github.com>
|
Hey @exploreriii , Ready for preview , please check. |
…ledger#773) Signed-off-by: Raja Rathour <imraja729@gmail.com>
exploreriii
left a comment
exploreriii
left a comment
There was a problem hiding this comment.
Hi @Raja-89 sorry the title of this issue was not titled correctly, but I'll persist as the description was correct.
Please edit:
examples/transfer_token.py
this is a file that already exists
so, you should keep the content in there and just modularise it
Currently, you have created a new file, token_trasnfer.py, which means we have duplicate content as have another at token_transfer.py
Thanks!
|
Hi @Raja-89 issues persist in this PR when you are able |
|
Hi @Raja-89 please confirm if you are able to continue working on this PRr |
Signed-off-by: Raja Rathour <imraja729@gmail.com>
Signed-off-by: Raja Rathour <193507075+Raja-89@users.noreply.github.com>
exploreriii
left a comment
exploreriii
left a comment
There was a problem hiding this comment.
Hi @Raja-89 very good just a changelog update please
CHANGELOG.md
Outdated
| - Limit workflow bot to one message per PR | ||
| - Refactored token-related example scripts (`token_delete.py`, `token_dissociate.py`, etc.) for improved readability and modularity. [#370] | ||
| - upgrade: step security action upgraded from harden-runner-2.13.1 to harden-runner-2.13.1 | ||
| - upgrade: step security action upgraded from harden-runner-2.13.1 to harden-runner. |
There was a problem hiding this comment.
could you make sure line 38 includes this:
- upgrade: step security action upgraded from harden-runner-2.13.1 to harden-runner-2.13.1
| - Added `examples/token_create_transaction_token_fee_schedule.py` to demonstrate creating tokens with custom fee schedules and the consequences of not having it. | ||
|
|
||
| ### Changed | ||
| - Upgraded step-security/harden-runner v2.13.2 |
There was a problem hiding this comment.
In line 34, please add your changelog entry about token transfer
…ledger#773) Signed-off-by: Raja Rathour <imraja729@gmail.com>
|
Hi @exploreriii , done the changes, ready for final review. |
examples/transfer_token.py
Outdated
|
|
||
| # Transfer Token | ||
| print("\nSTEP 4: Transfering Token...") | ||
| def transfer_transaction(client, operator_id, operator_key, recipient_id, token_id): |
There was a problem hiding this comment.
this should stay transfer_tokens() please
|
also rebase please :) |
Signed-off-by: Raja Rathour <imraja729@gmail.com>
Raja-89
force-pushed
the
feat/modular-token-transfer
branch
from
d9abfbb to
45a0dc4
Compare
|
@tech0priyanshu would you like to help review this PR? |
|
sure @exploreriii |
| uv run examples/transaction/transfer_transaction_fungible.py | ||
| python examples/transaction/transfer_transaction_fungible.py | ||
| uv run examples/transfer_token.py |
There was a problem hiding this comment.
Hi @Raja-89 , this file is now renamed to examples/transaction/transfer_transaction_fungible.py.
cc- @exploreriii
|
Also, please resolve the conflicts and rebase to pull in the latest changes from |
4 participants
Summary
Refactored
examples/token_transfer.pyinto modular functions for improved readability and structure.Introduced:
account_balance_query()transfer_transaction()main()entry point for clear script execution.Maintains identical runtime behavior and output.
Linked issue: #773