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

BumpFootprintExpiration operation #323

Merged
merged 5 commits into from
Aug 3, 2023

Conversation

Odraxs
Copy link
Contributor

@Odraxs Odraxs commented Aug 2, 2023

Context

To execute this operation and the RestoreFootprint op it was needed the files that allow building a SorobanTransactionData were before sending it to the simulate_transaction endpoint in addition to the BumpFootprintExpiration module for this pr.

Closes #314

Odraxs and others added 3 commits August 1, 2023 17:57
Co-authored-by: Edwin Steven Guayacan <EdwinGuayacan@users.noreply.github.com>
Co-authored-by: David Roldán <Odraxs@users.noreply.github.com>
Co-authored-by: Edwin Steven Guayacan <EdwinGuayacan@users.noreply.github.com>
@Odraxs Odraxs requested a review from miguelnietoa August 2, 2023 21:10
Copy link
Member

@miguelnietoa miguelnietoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Odraxs @EdwinGuayacan comments added

docs/examples/soroban/bump_footprint_expiration.md Outdated Show resolved Hide resolved
docs/examples/soroban/bump_footprint_expiration.md Outdated Show resolved Hide resolved
lib/tx_build/ledger/ledger_key_contract_code.ex Outdated Show resolved Hide resolved
lib/tx_build/ledger/ledger_key_contract_data.ex Outdated Show resolved Hide resolved
@Odraxs Odraxs requested a review from miguelnietoa August 3, 2023 13:28
Co-authored-by: Edwin Steven Guayacan <EdwinGuayacan@users.noreply.github.com>
@Odraxs Odraxs force-pushed the 314-bump-footprint-op branch from 7d2c913 to 12d933a Compare August 3, 2023 16:28
@Odraxs Odraxs merged commit 08a90fa into kommitters:v0.17 Aug 3, 2023
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