-
-
Notifications
You must be signed in to change notification settings - Fork 133
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/ add ETCswap V3 and V2 connectors #340
Changes from all commits
157d49c
91a61c0
0f461b5
14d66a9
d2c1841
7b417b0
bd9a268
60ff85e
b4bb002
64889f8
f53b584
3e48262
f175599
809dff7
e9a2e72
716ab08
4fa8093
fb11f32
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,126 @@ | ||
import abi from '../ethereum/ethereum.abi.json'; | ||
import { logger } from '../../services/logger'; | ||
import { Contract, Transaction, Wallet } from 'ethers'; | ||
import { EthereumBase } from '../ethereum/ethereum-base'; | ||
import { getEthereumConfig as getEthereumClassicChainConfig } from '../ethereum/ethereum.config'; | ||
import { Provider } from '@ethersproject/abstract-provider'; | ||
import { Chain as Ethereumish } from '../../services/common-interfaces'; | ||
import { ConfigManagerV2 } from '../../services/config-manager-v2'; | ||
import { EVMController } from '../ethereum/evm.controllers'; | ||
import { ETCSwapConfig } from '../../connectors/etcswap/etcswap.config'; | ||
|
||
export class EthereumClassicChain extends EthereumBase implements Ethereumish { | ||
private static _instances: { [name: string]: EthereumClassicChain }; | ||
private _chain: string; | ||
private _gasPrice: number; | ||
private _gasPriceRefreshInterval: number | null; | ||
private _nativeTokenSymbol: string; | ||
public controller; | ||
|
||
private constructor(network: string) { | ||
const config = getEthereumClassicChainConfig('ethereum-classic', network); | ||
super( | ||
'ethereum-classic', | ||
config.network.chainID, | ||
config.network.nodeURL, | ||
config.network.tokenListSource, | ||
config.network.tokenListType, | ||
config.manualGasPrice, | ||
config.gasLimitTransaction, | ||
ConfigManagerV2.getInstance().get('server.nonceDbPath'), | ||
ConfigManagerV2.getInstance().get('server.transactionDbPath'), | ||
); | ||
this._chain = config.network.name; | ||
this._nativeTokenSymbol = config.nativeCurrencySymbol; | ||
this._gasPrice = config.manualGasPrice; | ||
this._gasPriceRefreshInterval = | ||
config.network.gasPriceRefreshInterval !== undefined | ||
? config.network.gasPriceRefreshInterval | ||
: null; | ||
|
||
this.updateGasPrice(); | ||
this.controller = EVMController; | ||
} | ||
|
||
public static getInstance(network: string): EthereumClassicChain { | ||
if (EthereumClassicChain._instances === undefined) { | ||
EthereumClassicChain._instances = {}; | ||
} | ||
if (!(network in EthereumClassicChain._instances)) { | ||
EthereumClassicChain._instances[network] = new EthereumClassicChain( | ||
network, | ||
); | ||
} | ||
|
||
return EthereumClassicChain._instances[network]; | ||
} | ||
|
||
public static getConnectedInstances(): { | ||
[name: string]: EthereumClassicChain; | ||
} { | ||
return EthereumClassicChain._instances; | ||
} | ||
|
||
/** | ||
* Automatically update the prevailing gas price on the network from the connected RPC node. | ||
*/ | ||
async updateGasPrice(): Promise<void> { | ||
if (this._gasPriceRefreshInterval === null) { | ||
return; | ||
} | ||
|
||
const gasPrice: number = (await this.provider.getGasPrice()).toNumber(); | ||
|
||
this._gasPrice = gasPrice * 1e-9; | ||
|
||
setTimeout( | ||
this.updateGasPrice.bind(this), | ||
this._gasPriceRefreshInterval * 1000, | ||
); | ||
} | ||
|
||
// getters | ||
|
||
public get gasPrice(): number { | ||
return this._gasPrice; | ||
} | ||
|
||
public get nativeTokenSymbol(): string { | ||
return this._nativeTokenSymbol; | ||
} | ||
|
||
public get chain(): string { | ||
return this._chain; | ||
} | ||
|
||
// in place for mocking | ||
public get provider() { | ||
return super.provider; | ||
} | ||
|
||
getContract(tokenAddress: string, signerOrProvider?: Wallet | Provider) { | ||
return new Contract(tokenAddress, abi.ERC20Abi, signerOrProvider); | ||
} | ||
|
||
getSpender(reqSpender: string): string { | ||
let spender: string; | ||
if (reqSpender === 'etcswapLP') { | ||
spender = ETCSwapConfig.config.etcswapV3NftManagerAddress(this._chain); | ||
} else if (reqSpender === 'etcswap') { | ||
spender = ETCSwapConfig.config.etcswapV3SmartOrderRouterAddress( | ||
this._chain, | ||
); | ||
} else { | ||
spender = reqSpender; | ||
} | ||
return spender; | ||
} | ||
|
||
// cancel transaction | ||
async cancelTx(wallet: Wallet, nonce: number): Promise<Transaction> { | ||
logger.info( | ||
'Canceling any existing transaction(s) with nonce number ' + nonce + '.', | ||
); | ||
return super.cancelTxWithGasPrice(wallet, nonce, this._gasPrice * 2); | ||
} | ||
} |
This comment was marked as resolved.
Sorry, something went wrong. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have followed existing naming convention as the Uniswap connectors. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. okay thanks for clarification! great, i want it to follow the schema already present in the codebase |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,58 @@ | ||
import { AvailableNetworks } from '../../services/config-manager-types'; | ||
import { ConfigManagerV2 } from '../../services/config-manager-v2'; | ||
export namespace ETCSwapConfig { | ||
export interface NetworkConfig { | ||
allowedSlippage: string; | ||
gasLimitEstimate: number; | ||
ttl: number; | ||
maximumHops: number; | ||
etcswapV3SmartOrderRouterAddress: (network: string) => string; | ||
etcswapV3NftManagerAddress: (network: string) => string; | ||
etcswapV3FactoryAddress: (network: string) => string; | ||
quoterContractAddress: (network: string) => string; | ||
tradingTypes: (type: string) => Array<string>; | ||
chainType: string; | ||
availableNetworks: Array<AvailableNetworks>; | ||
useRouter?: boolean; | ||
feeTier?: string; | ||
} | ||
|
||
export const config: NetworkConfig = { | ||
allowedSlippage: ConfigManagerV2.getInstance().get( | ||
`etcswap.allowedSlippage` | ||
), | ||
gasLimitEstimate: ConfigManagerV2.getInstance().get( | ||
`etcswap.gasLimitEstimate` | ||
), | ||
ttl: ConfigManagerV2.getInstance().get(`etcswap.ttl`), | ||
maximumHops: ConfigManagerV2.getInstance().get(`etcswap.maximumHops`), | ||
etcswapV3SmartOrderRouterAddress: (network: string) => | ||
ConfigManagerV2.getInstance().get( | ||
`etcswap.contractAddresses.${network}.etcswapV3SmartOrderRouterAddress`, | ||
), | ||
etcswapV3NftManagerAddress: (network: string) => | ||
ConfigManagerV2.getInstance().get( | ||
`etcswap.contractAddresses.${network}.etcswapV3NftManagerAddress`, | ||
), | ||
etcswapV3FactoryAddress: (network: string) => | ||
ConfigManagerV2.getInstance().get( | ||
`etcswap.contractAddresses.${network}.etcswapV3FactoryAddress` | ||
), | ||
quoterContractAddress: (network: string) => | ||
ConfigManagerV2.getInstance().get( | ||
`etcswap.contractAddresses.${network}.etcswapV3QuoterV2ContractAddress` | ||
), | ||
tradingTypes: (type: string) => { | ||
return type === 'swap' ? ['AMM'] : ['AMM_LP']; | ||
}, | ||
chainType: 'EVM', | ||
availableNetworks: [ | ||
{ | ||
chain: 'ethereum-classic', | ||
networks: ['mainnet'] | ||
}, | ||
], | ||
useRouter: ConfigManagerV2.getInstance().get(`etcswap.useRouter`), | ||
feeTier: ConfigManagerV2.getInstance().get(`etcswap.feeTier`), | ||
}; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ethereum Classic's testnet is Mordor, not Goerli.
https://chainlist.org/chain/63
Recommended Funded Endpoint: https://rpc.mordor.etccooperative.org
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chris-mercer The point of replacing that url is to prevent calls to the default endpoint.
Workflows shouldn't access external services except in special cases.