Skip to content

Conversation

@calintje
Copy link
Contributor

This PR adds missing typedocs for the functions and types that are exported by the package.

It also:

  • removes export from getChainIdFromGenesisHash
  • adds Eclipse Testnet as a supported network, since the Whirlpool program is deployed there as well.

…rom getChainIdFromGenesisHash. Add eclipse-testnet as supported chain id.
@calintje calintje merged commit bb3eb77 into tx-sender Feb 20, 2025
3 of 4 checks passed
@calintje calintje deleted the calintje/typedocs-tx-sender branch February 20, 2025 23:23
parhim added a commit that referenced this pull request Feb 24, 2025
* somewhat stable scaffold without full functionality

* adds lookup tables to build tx

* changed global config and priority calculation (triton prop) & jito tips

* only calculate jito for solana

* bumped to web3.js 2.0 (in progress)

* send and confirm

* no sending signers

* support for lookup tables and legacy instructions

* changed structure

* signing capabilities

* updated dependencies

* docstrings and removes legacy conversion

* more exports

* refactored and tested with whirlpools instrucitons

* test suite init

* working decoder

* working tests

* added readme

* comments

* addresses most comments and updates tests

* a couple more refactors

* config refactors

* changed copy for config error

* confirmation logic and rewritten tests

* submedian for 50 percentile to reduce code complexity

* yarn lock

* ylock

* ran yarn format

* Typedocs for tx-sender (#763)

* Add missing typedocs to library exported functions. Remove 'export' from getChainIdFromGenesisHash. Add eclipse-testnet as supported chain id.

* Fix titan=triton. Add changeset

* Describe 50ema

* Update typedocs to reflect signature changes

* Update 50ema description

* added error catching for send tx and jito block engine url

* everything builds fine

* removed percentileNumber since jito 50ema is no longer passed in percentile

* trying to pass checks

---------

Co-authored-by: Calin <[email protected]>
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.

2 participants