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

feat(module): scaffold parameters for a module #1716

Merged
merged 30 commits into from
Oct 30, 2021
Merged

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Oct 23, 2021

close #1699

What does this PR?

Add in the starport s module command the flag --params that allow specifying the param fields of the module.

How to test?

starport s module launch --params minLaunch:uint,maxLaunch:int

@Pantani Pantani self-assigned this Oct 23, 2021
Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

👍

The command works fine for me

@Pantani Pantani requested a review from lumtis October 26, 2021 17:01
lumtis
lumtis previously approved these changes Oct 27, 2021
Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

💯

@ilgooz ilgooz merged commit 25dbd8b into develop Oct 30, 2021
@ilgooz ilgooz deleted the feat/add-params branch October 30, 2021 16:01
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
* add params structure

* add missing params file

* add testutil/keeper

* add integration test for params

* fix ibc testutil keeper

* add params test for ibc

* use types.DefaultParams for TestParamsQuery

* fix identantion for plush files

* add param tests for keeper

* fix subspace not found

* fix unused todo var

* fix param subspace for testutil

* fix unused import for module without params

* fix condition with wrong syntax

* fix integration test collision

* fix dummy value index

* fix dummy ValueInvalidIndex

* value index should be a string not a number

* prevent unused import error cor strconv

* avoid use strconv for string params types

* fix param index for the genesis state and some comments

* add bool param into the integration test

* improve todo comment for default value

Co-authored-by: İlker G. Öztürk <[email protected]>
Co-authored-by: Lucas Bertrand <[email protected]>
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.

Scaffold parameters for a module
3 participants