Skip to content
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.

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

Clarify what app_args should be #342

Open
avislash opened this issue Mar 19, 2021 · 1 comment
Open

Clarify what app_args should be #342

avislash opened this issue Mar 19, 2021 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@avislash
Copy link

avislash commented Mar 19, 2021

The current state
The Python snippet of the update section of https://developer.algorand.org/docs/features/asc1/stateful/sdks/#update has the following line

txn = txn = transaction.ApplicationUpdateTxn(sender, params, app_id, \ approval_program, clear_program, app_args)

The app_args parameter is not defined prior to this snippet.

The proposed state
Define what app_args should be in the line right before it is used

For example
app_args = [ ]
txn = txn = transaction.ApplicationUpdateTxn(sender, params, app_id, \ approval_program, clear_program, app_args)
Provide any additional context
Add any other context or screenshots about the proposed update here.

@avislash avislash added the documentation Improvements or additions to documentation label Mar 19, 2021
@iskysun96
Copy link
Collaborator

@avislash Do you think its necessary to define all the other arguments as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants
@avislash @iskysun96 and others