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/remove default values #109

Closed
wants to merge 68 commits into from

Conversation

faust403
Copy link
Member

@faust403 faust403 commented Jul 3, 2024

No description provided.

@faust403 faust403 force-pushed the feat/remove-default-values branch from 7aa282b to c01e934 Compare July 4, 2024 09:34
Comment on lines 172 to 177
"lsm_redeem_threshold":2,
"lsm_min_bond_amount":"1",
"lsm_redeem_max_interval":60000,
"bond_limit":"0",
"min_stake_amount":"2",
"icq_update_delay": 5
Copy link
Member

Choose a reason for hiding this comment

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

Please replace these default values with variables as well

},
"base_denom":"'"$uatom_on_neutron_denom"'",
"core_params":{
"idle_min_interval":60,
Copy link
Member

Choose a reason for hiding this comment

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

This one as well

"transfer_channel_id":"'"$NEUTRON_SIDE_TRANSFER_CHANNEL_ID"'",
"port_id":"transfer",
"denom":"'"$TARGET_BASE_DENOM"'",
"update_period":100
Copy link
Member

Choose a reason for hiding this comment

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

here

Comment on lines 160 to 164
"description":"Drop token",
"display":"drop",
"exponent":6,
"name":"Drop liquid staking token",
"symbol":"DROP"
Copy link
Member

Choose a reason for hiding this comment

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

This values as well, but set default value to the current constant

Comment on lines 180 to 181
"min_stake_amount":"10000",
"min_ibc_transfer":"10000",
Copy link
Member

Choose a reason for hiding this comment

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

and here

…ate.bash and update .env.instantiate.example
@oldremez oldremez changed the base branch from feat/deploy-script to feat/audit-fixes July 9, 2024 08:17
@faust403 faust403 requested a review from albertandrejev July 9, 2024 08:41
@faust403 faust403 closed this Jul 10, 2024
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.

5 participants