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

Allow bumping other instances #1148

Merged
merged 2 commits into from
Nov 13, 2023
Merged

Allow bumping other instances #1148

merged 2 commits into from
Nov 13, 2023

Conversation

sisuresh
Copy link
Contributor

What

Add SDK function for extend_contract_instance_and_code. I think the existing extend_ttl function should be called extend_current_ttl, and this new SDK function would be extend_ttl, but I'm not sure if it's worth updating now.

@leighmcculloch
Copy link
Member

I think the existing extend_ttl function should be called extend_current_ttl, and this new SDK function would be extend_ttl, but I'm not sure if it's worth updating now.

I'm not convinced about this, because otherwise we'd be attaching the term "current" to all the functions in the storage types.

@sisuresh
Copy link
Contributor Author

I think the existing extend_ttl function should be called extend_current_ttl, and this new SDK function would be extend_ttl, but I'm not sure if it's worth updating now.

I'm not convinced about this, because otherwise we'd be attaching the term "current" to all the functions in the storage types.

My reasoning for this was due to the fact that the host functions are called extend_current_contract_instance_and_code and update_current_contract_wasm. Maybe the names would make more sense if we removed "current" and distinguished the ones that operate on other contracts.

leighmcculloch
leighmcculloch previously approved these changes Nov 13, 2023
soroban-sdk/src/storage.rs Outdated Show resolved Hide resolved
soroban-sdk/src/storage.rs Outdated Show resolved Hide resolved
soroban-sdk/src/storage.rs Outdated Show resolved Hide resolved
@leighmcculloch leighmcculloch dismissed their stale review November 13, 2023 19:34

Because changes are needed to move the fn to deployer.

Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

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

One suggestion, regardless 👍🏻.

soroban-sdk/src/deploy.rs Outdated Show resolved Hide resolved
Co-authored-by: Leigh McCulloch <351529+leighmcculloch@users.noreply.github.com>
@sisuresh sisuresh enabled auto-merge November 13, 2023 20:11
@sisuresh sisuresh added this pull request to the merge queue Nov 13, 2023
Merged via the queue into stellar:main with commit ae2d25e Nov 13, 2023
@sisuresh sisuresh deleted the extend branch November 13, 2023 21:11
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