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

Add interchain Starship support #160

Closed
wants to merge 196 commits into from
Closed

Add interchain Starship support #160

wants to merge 196 commits into from

Conversation

CyberHoward
Copy link
Contributor

Adds the ability to connect to Starship infrastructure, deploy to it and create channels on it.

@Kayanski
Copy link
Contributor

Kayanski commented Aug 1, 2023

In order to continue development, I merged
#172 and #152 into the branch

@CyberHoward CyberHoward mentioned this pull request Aug 17, 2023
@@ -106,7 +106,7 @@ impl TxBuilder {
sim_gas_used as f64 * GAS_BUFFER
};
let fee_amount = gas_expected
* (wallet.daemon_state.chain_data.fees.fee_tokens[0].fixed_min_gas_price + 0.00001);
* (wallet.daemon_state.chain_data.fees.fee_tokens[0].average_gas_price + 0.00001);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The problem with this is that the const configs that we have set the min_gas_price with the price that we set.

We could start using chain_registry by default?

@Kayanski
Copy link
Contributor

Kayanski commented Sep 1, 2023

Fixes #6

@Kayanski
Copy link
Contributor

Kayanski commented Sep 3, 2023

Closed source starship support, closing this PR

@Kayanski Kayanski closed this Sep 3, 2023
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