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

extend ttl should use readOnly #1179

Merged
merged 1 commit into from
Jan 13, 2025
Merged

extend ttl should use readOnly #1179

merged 1 commit into from
Jan 13, 2025

Conversation

jeesunikim
Copy link
Contributor

@jeesunikim jeesunikim commented Jan 13, 2025

the simulate deep dive example was using setReadWrite instead of setReadOnly for extend ttl operation (sdk doc)

Builds an operation to bump the time-to-live (TTL) of the ledger keys. The keys for extension have to be provided in the read-only footprint of the transaction.

@stellar-jenkins
Copy link

@jeesunikim jeesunikim merged commit 4fb38cc into main Jan 13, 2025
2 checks passed
@jeesunikim jeesunikim deleted the update-ttl-readoptions branch January 13, 2025 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants