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

Release v0.95.1 #104

Closed
wants to merge 426 commits into from
Closed

Release v0.95.1 #104

wants to merge 426 commits into from

Conversation

bleaker0x
Copy link
Contributor

@bleaker0x bleaker0x commented Apr 14, 2022

What's new?

Liquidity Pools

Chains

Tokens

New addresses for existing tokens

  • Cronos
  • Optimism

Features, functions, fixes, and more

  • ci

    • GPG-signing of release tarballs for GitHub releases (be124af)
    • Implement GitHub actions workflow for developement/alpha/beta release publishing (c3e864c)
    • Change "target" field of tsconfig.json to "ES5", enabling greater backwards-compatibility with older NodeJS versions and various build systems (2ac1627)
    • Coverage now ignores typegen code in internal/gen (5af24cc)
  • bridge

    • Implement checkBridgeTransactionComplete in SynapseBridge namespace, enabling basic checking of bridge transaction status (b444b15)
    • Implement AvaxJewelMigration contract and "bridging" (11b3834)
    • Implement class wrapper for BridgeConfigV3. It makes life easier (d2d2003)
  • common/networks

    • Add bridgeableTokens getter function to Network class (179d702)
  • tokenswap

    • Implement functions for adding and removing liquidity from Liquidity Pools -- addLiquidity, calculateAddLiquidity, removeLiquidity, calculateRemoveLiquidity, removeLiquidityOneToken, calculateRemoveLiquidityOneToken (6e230fd, 57e6fdb, caa4b5d)
    • Filter out wrapped gas tokens from detailedTokenSwapMap() result (8b9c696)
    • Refactor UnsupportedSwapError type into an extension of inbuilt Error type (cc645f5)
  • entities

    • implement connection "cacheing" (more like reuse) for connections to contracts such as SynapseBridge, L2BridgeZap, etc. (7359c56)
  • tokens

    • Add gasTokenForChain function to Tokens namespace, which returns the native currency token object for the passed Chain ID -- if it exists. (7d9b47d)
  • token

  • chainid

    • Implement function for checking if a (known) Chain ID supports EIP-1559 (5aa1535)
  • networks

    • Add supportsEIP1559 property to Networks class. This property is a boolean reflecting whether a network supports EIP-1559 (067b75d)
    • Add chainCurrencyCoingeckoId property to Network class. For chains whose chain currencies have CoinGecko ids, this property provides that id.
  • common/gasoptions

    • Implement setup enabling SDK consumers to provide their own gas limits/fees/prices to various transaction builders/executors (83377e8)

Refactors

  • common/chainid, internal/swaptype
    • Refactor type of ChainId and SwapType from enum to const object using as const syntax, enabling better control of inputs, parameters, and types. (3094359, 2afa767)

Testing improvements

Thank you @BlazeWasHere for adding several new test cases to the test suites:

  • Better random value generation (37cd132)
  • Improved valueToWei testing (00ab02e)
  • Extra testing for Cronos (4ae4cba)
  • Tokens.TokenFromSymbol test cases (82d395d)
  • Liquidity amounts map tests (7da69b7)

Additionally, more test cases for various smaller parts have been implemented.

@bleaker0x bleaker0x added enhancement New feature or request help wanted Extra attention is needed new-network Changesets which include new networks tests Changesets which add more or improve existing tests new token Changesets which include new tokens build Issues and PRs relating to build/compilation labels Apr 14, 2022
@coveralls
Copy link
Collaborator

coveralls commented Apr 14, 2022

Coverage Status

Coverage increased (+0.1%) to 99.637% when pulling b47bd0c on release.v0.95.0 into 0f9fbc5 on master.

@bleaker0x bleaker0x force-pushed the release.v0.95.0 branch 2 times, most recently from 58f88a4 to 00b2f9a Compare April 15, 2022 00:04
package.json Outdated Show resolved Hide resolved
src/common/utils.ts Outdated Show resolved Hide resolved
- token: add coingeckoId field to Token interface
- tokens: add coingeckoId constructor params to tokens with known ids
  (per #120)
For chains whose chain currencies have CoinGecko ids, provide that
value.
@bleaker0x bleaker0x removed their assignment Aug 5, 2022
@trajan0x
Copy link
Contributor

Thanks for the contribution, but this repo is now deprecated and remaining issues being closed. The next iteration of the sdk can be found here

@trajan0x trajan0x closed this Apr 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build Issues and PRs relating to build/compilation enhancement New feature or request help wanted Extra attention is needed new token Changesets which include new tokens new-network Changesets which include new networks tests Changesets which add more or improve existing tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

support for coingecko ids Unique symbols for tokens
6 participants