docs: update tutorials to work with SDK v0.50
#3705
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
component:ci
|
Visit the preview URL for this PR (updated for commit e1e86e5): https://igntservices-docs--pr3705-fix-docs-sdk-v0-50-0w3fgijv.web.app (expires Sat, 18 Nov 2023 11:59:58 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 95379efd94dd497aaa37c2d0354e6e2cafca5ec5 |
julienrbrt
approved these changes
Oct 19, 2023
| @@ -3,7 +3,7 @@ | |||
| ## Create a blog post by Alice | |||
|
|
|||
| ``` | |||
| blogd tx blog create-post hello world --from alice | |||
| blogd tx blog create-post hello world --from alice --chain-id blog | |||
There was a problem hiding this comment.
Why precising always the chain id? Maybe we should set it in the client.toml first.
blogd config set client chain-id blog
There was a problem hiding this comment.
The queries/txs are not working without the chain-id into the SDK v0.50, and are already set the chain id into the config
There was a problem hiding this comment.
Really? That sounds like a bug in the SDK to me.
Are you aware of any other flag like that?
I know there are flag related issues in the SDK right now (cosmos/cosmos-sdk#17822 / cosmos/cosmos-sdk#18122) which I am working on. I thought it was only --node.
Ehsan-saradar
approved these changes
Oct 23, 2023
@Pantani @julienrbrt if we plan to merge these two PRs after the new Ignite CLI release we could merge this one right after #3659 is merged. |
Pantani
force-pushed
the
feat/sdk-v0.50
branch
2 times, most recently
from
cc7bd5e to
3a8e27d
Compare
Pantani
added a commit
that referenced
this pull request
Nov 20, 2023
* update go.sum * cosmos-sdk v0.50 imports * changes imports and update sdk * fix vet issues * fix unused imports * bump ibc * add changelog * update mocks * rollback some test changes * fix go version for the git workflows * fix wront tx result for abci tx * fix wrong test case * remove some legacy methods and add module import blank to the `app_config.go` * remove legacy amino and fix genesis commands * fix module orders * remove unused modules and commands * fix module imports * use statstore instead key and memory store * fix missing runtime import for test keepers * remove typo * fix missing imports * add msg signer option to the tx proto messages * add release branch to trigger tests * remove invalid test case * fix typo * fix missing create mesage option for single messages scaffold * Initialized with Ignite CLI * remove unused files * `--events` flag was moved to `--query` into `query tx` command * refactor: v0.50 store app wiring (#3672) * refactor: v0.50 store app wiring * updates * updates --------- Co-authored-by: Danilo Pantani <[email protected]> * refactor: remove unecessary boilerplate in module (#3673) * refactor: remove unecessary boilerplate in module * updates --------- Co-authored-by: Danilo Pantani <[email protected]> * refactor: simplify app.go (#3675) * refactor: simplify app.go * updates --------- Co-authored-by: Danilo Pantani <[email protected]> * fix simulation types * fix query txs params * remove some empty lines * refactor: remove unecessary boilerplate (#3676) Co-authored-by: Danilo Pantani <[email protected]> * update go.mod.plush * rollback go.mod changes * fix module discovery to use the keepers * bump sdk and ibc * find module per keeper * check blank imports * remove unused test data * remove old support methods and move module pkg to module folder * fix the module scaffold to find the module folder * add more test case and blank imports * use basepath instead roothopath * add pre blockers * fix cmd root_v2 * change ProvideClientContext to a pointer * remove unused unit tests and fix others * bump cosmos-sdk and run make format * fix root.go * fix wrong tx swagger generation * fix ibc module import * fix unit tests * fix app config for the unit tests * return ModuleCDC for ibc modules * fix capability keeper without app wiring * cleanup root.go and app.go * rollback txConfigOpts * use keepers injections as optional * fix error handler assign for sim tests * fix root sdk config call * skip lint cache * fix unit tests for app.go discovery * fix test data * add mmising interface methods for some tests * fix cosmosanalystis find implementation methods * fix ibc keepers for the testutil pkg * fix expected keepers injection conflict * remove unused expected keepers interfsces * fix capability keeper nullpointer for unit tests * fix module_ibc.go scaffolder * rollback sdk context * add creator options for packet msgs * bump cosmos-sdk * fix unhandled error * rollback lint version and improve app module comments * move appconfig inject to app.go * fix default test config * increase chain test timeout for integration * remove unused `go.sum` files * bump cosmos-sdk and ibc * fix network tests * add generic clenaup for the network config * add cfg addresses manually * avoid override config ports * add chain id and log format to tx integration test * add tx output * add json output for key query * bump ibc/capability and sdk/client * bum go dependencies * rollback not linux support dependency * bump ibc * regenerate mocks * docs: update tutorials to work with SDK `v0.50` (#3705) * chore: add `--chain-id` flag to blog commands * chore: use `cosmossdk.io/store` * chore: change blog tutorial to use the store service * chore: use `cosmossdk.io/errors` * chore: change "PostID" to "PostId" * chore: add missing import to code snippet * fix: correct expected keepers interface args * chore: add `--chain-id` flag to loan commands * ci: fix broken links * ci: fix broken references * ci: fix broken reference * chore: fix typo --------- Co-authored-by: Danilo Pantani <[email protected]> * bum chain ibc * fix: fix ibc registration in 0.50 (#3758) * fix: fix ibc registration in 0.59 * updates * nit --------- Co-authored-by: Danilo Pantani <[email protected]> * register capability module and fix genesis order * ibc params register --------- Co-authored-by: Pantani <Pantani> Co-authored-by: Developer Experience team at Ignite <[email protected]> Co-authored-by: Julien Robert <[email protected]> Co-authored-by: Jerónimo Albi <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.