Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

Update outdated code. #759

Merged
merged 1 commit into from
Mar 1, 2024

Conversation

overcat
Copy link
Contributor

@overcat overcat commented Feb 29, 2024

Here are some Python and JavaScript codes that have become outdated and cannot run smoothly under the latest version of SDK. This PR updates them.

@stellar-jenkins
Copy link

1 similar comment
@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@ElliotFriend
Copy link
Contributor

@overcat This is an incredibly helpful contribution!!! We truly can't thank you enough for this! Are the changes ready to be merged, or is there still some work (or other updates) you have left?

I'm happy to approve and merge this, just want to make sure I don't do it while you're still in the middle of it 👍🏻

Thanks again!

@overcat
Copy link
Contributor Author

overcat commented Mar 1, 2024

Hi @ElliotFriend, this PR has been completed, and all the code has been tested by me.

But there is one part that I did not modify, which is that the increment contract has changed. Now the COUNTER in the increment contract is stored in the instance, which does not meet the requirements of the example very well. So I made some modifications to this contract, I store COUNTER in Persistent. All these examples are based on this point.

See https://github.com/stellar/soroban-examples/blob/main/increment/src/lib.rs#L21

@ElliotFriend
Copy link
Contributor

Yeah, I think I came across something that threw me off recently with COUNTER being in instance storage now. Thanks for working around it.

Thanks again! Big help here!

@ElliotFriend ElliotFriend merged commit e86caa3 into stellar-deprecated:main Mar 1, 2024
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants