-
Notifications
You must be signed in to change notification settings - Fork 894
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
chore: update for Preview 20 #138
base: main
Are you sure you want to change the base?
Conversation
89c691d
to
6b4cea5
Compare
- 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`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! One small dep nitpick
@@ -28,7 +28,7 @@ | |||
"next": "^13.4.4", | |||
"react": "^18.2.0", | |||
"react-dom": "^18.2.0", | |||
"soroban-client": "1.0.0-beta.2" | |||
"stellar-sdk": "^11.1.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you use @stellar/stellar-sdk
, instead? 🙏
@@ -14,6 +13,9 @@ type PagingKey = string | |||
*/ | |||
const paging: Record<PagingKey, { lastLedgerStart?: number, pagingToken?: string }> = {} | |||
|
|||
// `EventResponse` is not exported from stellar-sdk | |||
type EventResponse = SorobanRpc.Api.GetEventsResponse['events'][number] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
noted! will fix :o
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will be fixed by stellar/js-stellar-sdk#904
abundance
source fromtoken
contract at Changes for the next release soroban-examples#277. Note that I did not author most of the changes in thecontracts/abundance
folder. I only copied in the changes originally authored in thesoroban-examples
repo, then updated for the single tweak that thisabundance
contract already had: 1. comments on themint
method; 2. slightly modifiedmint
behavior.AssembledTransaction
, which has aresult
getter and asignAndSend
method, as explained in feat(bindings-ts)!: API change: returnAssembledTransaction
s, require signAndSend, and support multi-auth workflows stellar-cli#1034stellar-sdk
rather thansoroban-client
@stellar/freighter-api