Skip to content
This repository has been archived by the owner on Apr 11, 2023. It is now read-only.

feat(investor-foxy): rename amountDesired -> amountDesiredCryptoBaseUnit #1131

Draft
wants to merge 2 commits into
base: feat_investor_foxy_ethers_js_batch_provider
Choose a base branch
from

Conversation

gomesalexandre
Copy link
Contributor

@gomesalexandre gomesalexandre commented Dec 8, 2022

Something something explicit over implicit

Copy link
Contributor Author

@gomesalexandre gomesalexandre force-pushed the feat_investor_foxy_ethers_js_batch_provider branch from de36c42 to 65dbd5f Compare December 9, 2022 14:58
@gomesalexandre gomesalexandre force-pushed the feat_foxy_rename_amountDesired_amountDesiredCryptoBaseUnit branch from be49931 to 5c89623 Compare December 9, 2022 14:58
@@ -281,15 +281,19 @@ export class FoxyApi {
}

async estimateAddLiquidityGas(input: EstimateGasTxInput): Promise<string> {
const { amountDesired, userAddress, contractAddress } = input
const {
amountDesiredCryptoBaseUnit: amountDesiredCryptoBaseUnit,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

lsp pls

@gomesalexandre gomesalexandre force-pushed the feat_investor_foxy_ethers_js_batch_provider branch from 65dbd5f to 7a69378 Compare December 31, 2022 00:33
@gomesalexandre gomesalexandre force-pushed the feat_foxy_rename_amountDesired_amountDesiredCryptoBaseUnit branch from 5c89623 to 7d937ee Compare December 31, 2022 00:34
Copy link
Member

@0xApotheosis 0xApotheosis left a comment

Choose a reason for hiding this comment

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

🏅

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants