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

Changes for the next release #277

Merged
merged 6 commits into from
Sep 18, 2023
Merged

Changes for the next release #277

merged 6 commits into from
Sep 18, 2023

Conversation

leighmcculloch
Copy link
Member

@leighmcculloch leighmcculloch commented Sep 1, 2023

What

All changes for the next release.

Why

To prepare and group changes without merging since developers are reading the docs and looking at the main branch, we don't want to introduce changes for the next release until the next release is out.

cc @sisuresh @dmkozh @tsachiherman

Merging

When it comes time to merge this pull request, it is ideal if an admin can turn off requiring merging via squash, and then the PR is merged with the merge commit strategy.

@leighmcculloch leighmcculloch marked this pull request as draft September 1, 2023 23:43
@leighmcculloch leighmcculloch mentioned this pull request Sep 1, 2023
@sreuland
Copy link

sreuland commented Sep 6, 2023

@sisuresh , this pr/branch is serving same purpose of latest pre-release as https://github.com/stellar/soroban-examples/releases/tag/testnet was, I think can reference this going forward and retire the former?

@leighmcculloch, will this get bumped to latest sdk from stellar/rs-soroban-sdk#1081 or hold for other? the contracts are used by system-test and working on getting them to pass with latest refs, tools/#928

@sisuresh
Copy link
Contributor

sisuresh commented Sep 6, 2023

@sisuresh , this pr/branch is serving same purpose of latest pre-release as https://github.com/stellar/soroban-examples/releases/tag/testnet was, I think can reference this going forward and retire the former?

@leighmcculloch, will this get bumped to latest sdk from stellar/rs-soroban-sdk#1081 or hold for other? the contracts are used by system-test and working on getting them to pass with latest refs, tools/#928

Yeah we should retire the tag if possible since that adds an extra step in the process for us.

#271 updates this PR, but not to stellar/rs-soroban-sdk#1081. I can get to this tomorrow, but @leighmcculloch if you have time to address this today go ahead.

@sisuresh sisuresh marked this pull request as ready for review September 18, 2023 17:55
@sisuresh sisuresh enabled auto-merge (squash) September 18, 2023 20:07
@sisuresh sisuresh merged commit fed66d2 into main Sep 18, 2023
77 checks passed
@sisuresh sisuresh deleted the release/next branch September 18, 2023 20:07
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Sep 20, 2023
- Pin soroban-cli to latest commit from
  stellar/stellar-cli#973. This PR will be
  mergeable once a new quickstart image is released and we can verify
  that its tests pass.
- Update the contract's SDK version
  - Copy in abundance source from token contract at "Changes for the next
    release," stellar/soroban-examples#277. Note
    that I did not author most of the changes in the contracts/abundance
    folder. I only copied in the changes originally authored by
    @leighmcculloch, then updated for the single tweak that this
    abundance contract already had: 1. comments on the mint method; and
    2.  slightly modified mint behavior.
- New CLI's typescript-bindings-generated libraries export a `Contract`
  class, rather than a flat bag of functions. Instantiating these
  contracts is now taken care of in `shared/contracts.ts`, with code
  throughout the rest of the codebase referencing these instances.
- Instantiating these classes required access to the `network` and
  `rpcUrl` used with the `initialize.sh` script. NextJS doesn't allow
  reading the files we already write (like `.soroban-example-dapp/network`),
  so for now I've also `echo`d relevant values to a `shared/config.json`
  file, which is hidden. This could probably be cleaned up in the future
  by putting all relevant settings in an `.env` file, and cleaning up
  the package.json scripts to use `dotenv cross-env` or similar.
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Sep 20, 2023
- Pin soroban-cli to latest commit from
  stellar/stellar-cli#973. This PR will be
  mergeable once a new quickstart image is released and we can verify
  that its tests pass.
- Update the contract's SDK version
  - Copy in abundance source from token contract at "Changes for the next
    release," stellar/soroban-examples#277. Note
    that I did not author most of the changes in the contracts/abundance
    folder. I only copied in the changes originally authored by
    @leighmcculloch, then updated for the single tweak that this
    abundance contract already had: 1. comments on the mint method; and
    2.  slightly modified mint behavior.
- New CLI's typescript-bindings-generated libraries export a `Contract`
  class, rather than a flat bag of functions. Instantiating these
  contracts is now taken care of in `shared/contracts.ts`, with code
  throughout the rest of the codebase referencing these instances.
- Instantiating these classes required access to the `network` and
  `rpcUrl` used with the `initialize.sh` script. NextJS doesn't allow
  reading the files we already write (like `.soroban-example-dapp/network`),
  so for now I've also `echo`d relevant values to a `shared/config.json`
  file, which is hidden. This could probably be cleaned up in the future
  by putting all relevant settings in an `.env` file, and cleaning up
  the package.json scripts to use `dotenv cross-env` or similar.
maxymkotko added a commit to maxymkotko/Soroban-dApp that referenced this pull request Dec 13, 2023
- Pin soroban-cli to latest commit from
  stellar/stellar-cli#973. This PR will be
  mergeable once a new quickstart image is released and we can verify
  that its tests pass.
- Update the contract's SDK version
  - Copy in abundance source from token contract at "Changes for the next
    release," stellar/soroban-examples#277. Note
    that I did not author most of the changes in the contracts/abundance
    folder. I only copied in the changes originally authored by
    @leighmcculloch, then updated for the single tweak that this
    abundance contract already had: 1. comments on the mint method; and
    2.  slightly modified mint behavior.
- New CLI's typescript-bindings-generated libraries export a `Contract`
  class, rather than a flat bag of functions. Instantiating these
  contracts is now taken care of in `shared/contracts.ts`, with code
  throughout the rest of the codebase referencing these instances.
- Instantiating these classes required access to the `network` and
  `rpcUrl` used with the `initialize.sh` script. NextJS doesn't allow
  reading the files we already write (like `.soroban-example-dapp/network`),
  so for now I've also `echo`d relevant values to a `shared/config.json`
  file, which is hidden. This could probably be cleaned up in the future
  by putting all relevant settings in an `.env` file, and cleaning up
  the package.json scripts to use `dotenv cross-env` or similar.
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Dec 18, 2023
_work in progress: needs CLI v20.0.3, which must actually include latest bindings logic. Somehow 20.0.2 does not, even though it was tagged post-multi-auth merge https://github.com/stellar/soroban-tools/commits/main/)._

- Pin soroban-cli to latest version, 20.0.3
- Update the contract's SDK version
  - **Copy in `abundance` source from `token` contract** at stellar/soroban-examples#277. Note that I **did not author most of the changes in the `contracts/abundance` folder**. I _only_ copied in the changes originally authored in the `soroban-examples` repo, then updated for the single tweak that this `abundance` contract already had: 1. comments on the `mint` method; 2. slightly modified `mint` behavior.
- New CLI's typescript-bindings-generated libraries now return an `AssembledTransaction`, which has a `result` getter and a `signAndSend` method, as explained in stellar/stellar-cli#1034
- Use `stellar-sdk` rather than `soroban-client`
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Dec 20, 2023
- Pin soroban-cli to latest version, 20.0.3
- Update the contract's SDK version
  - **Copy in `abundance` source from `token` contract** at stellar/soroban-examples#277. Note that I **did not author most of the changes in the `contracts/abundance` folder**. I _only_ copied in the changes originally authored in the `soroban-examples` repo, then updated for the single tweak that this `abundance` contract already had: 1. comments on the `mint` method; 2. slightly modified `mint` behavior.
- New CLI's typescript-bindings-generated libraries now return an `AssembledTransaction`, which has a `result` getter and a `signAndSend` method, as explained in stellar/stellar-cli#1034
- Use `stellar-sdk` rather than `soroban-client`
- Use latest `@stellar/freighter-api`
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Dec 20, 2023
- Pin soroban-cli to latest version, 20.1.0
- Update the contract's SDK version
  - **Copy in `abundance` source from `token` contract** at stellar/soroban-examples#277. Note that I **did not author most of the changes in the `contracts/abundance` folder**. I _only_ copied in the changes originally authored in the `soroban-examples` repo, then updated for the single tweak that this `abundance` contract already had: 1. comments on the `mint` method; 2. slightly modified `mint` behavior.
- New CLI's typescript-bindings-generated libraries now return an `AssembledTransaction`, which has a `result` getter and a `signAndSend` method, as explained in stellar/stellar-cli#1034
- Use `stellar-sdk` rather than `soroban-client`
- Use latest `@stellar/freighter-api`
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Dec 21, 2023
- Pin soroban-cli to latest version, 20.1.0
- Update the contract's SDK version
  - **Copy in `abundance` source from `token` contract** at stellar/soroban-examples#277. Note that I **did not author most of the changes in the `contracts/abundance` folder**. I _only_ copied in the changes originally authored in the `soroban-examples` repo, then updated for the single tweak that this `abundance` contract already had: 1. comments on the `mint` method; 2. slightly modified `mint` behavior.
- New CLI's typescript-bindings-generated libraries now return an `AssembledTransaction`, which has a `result` getter and a `signAndSend` method, as explained in stellar/stellar-cli#1034
- Use `stellar-sdk` rather than `soroban-client`
- Use latest `@stellar/freighter-api`
chadoh added a commit to AhaLabs/soroban-example-dapp that referenced this pull request Jan 5, 2024
- Pin soroban-cli to latest version, 20.1.1
- Update the contract's SDK version
  - **Copy in `abundance` source from `token` contract** at stellar/soroban-examples#277. Note that I **did not author most of the changes in the `contracts/abundance` folder**. I _only_ copied in the changes originally authored in the `soroban-examples` repo, then updated for the single tweak that this `abundance` contract already had: 1. comments on the `mint` method; 2. slightly modified `mint` behavior.
- New CLI's typescript-bindings-generated libraries now return an `AssembledTransaction`, which has a `result` getter and a `signAndSend` method, as explained in stellar/stellar-cli#1034
- Use `stellar-sdk` rather than `soroban-client`
- Use latest `@stellar/freighter-api`
spotlight301 pushed a commit to spotlight301/Dapp-Soro that referenced this pull request Sep 30, 2024
- Pin soroban-cli to latest commit from
  stellar/stellar-cli#973. This PR will be
  mergeable once a new quickstart image is released and we can verify
  that its tests pass.
- Update the contract's SDK version
  - Copy in abundance source from token contract at "Changes for the next
    release," stellar/soroban-examples#277. Note
    that I did not author most of the changes in the contracts/abundance
    folder. I only copied in the changes originally authored by
    @leighmcculloch, then updated for the single tweak that this
    abundance contract already had: 1. comments on the mint method; and
    2.  slightly modified mint behavior.
- New CLI's typescript-bindings-generated libraries export a `Contract`
  class, rather than a flat bag of functions. Instantiating these
  contracts is now taken care of in `shared/contracts.ts`, with code
  throughout the rest of the codebase referencing these instances.
- Instantiating these classes required access to the `network` and
  `rpcUrl` used with the `initialize.sh` script. NextJS doesn't allow
  reading the files we already write (like `.soroban-example-dapp/network`),
  so for now I've also `echo`d relevant values to a `shared/config.json`
  file, which is hidden. This could probably be cleaned up in the future
  by putting all relevant settings in an `.env` file, and cleaning up
  the package.json scripts to use `dotenv cross-env` or similar.
joaovmp pushed a commit to joaovmp/Soroban_dapp that referenced this pull request Oct 23, 2024
- Pin soroban-cli to latest commit from
  stellar/stellar-cli#973. This PR will be
  mergeable once a new quickstart image is released and we can verify
  that its tests pass.
- Update the contract's SDK version
  - Copy in abundance source from token contract at "Changes for the next
    release," stellar/soroban-examples#277. Note
    that I did not author most of the changes in the contracts/abundance
    folder. I only copied in the changes originally authored by
    @leighmcculloch, then updated for the single tweak that this
    abundance contract already had: 1. comments on the mint method; and
    2.  slightly modified mint behavior.
- New CLI's typescript-bindings-generated libraries export a `Contract`
  class, rather than a flat bag of functions. Instantiating these
  contracts is now taken care of in `shared/contracts.ts`, with code
  throughout the rest of the codebase referencing these instances.
- Instantiating these classes required access to the `network` and
  `rpcUrl` used with the `initialize.sh` script. NextJS doesn't allow
  reading the files we already write (like `.soroban-example-dapp/network`),
  so for now I've also `echo`d relevant values to a `shared/config.json`
  file, which is hidden. This could probably be cleaned up in the future
  by putting all relevant settings in an `.env` file, and cleaning up
  the package.json scripts to use `dotenv cross-env` or similar.
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