[ FIX ] Fix bugs for #1837 and enable feePayer again #1981
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…er and WhileValidTxSender to ensure correct fee payer assignment and transaction handling This commit introduces a new test script that validates the functionality of the RetryTxSender and WhileValidTxSender classes. It verifies that transactions are sent correctly with the appropriate fee payer, both when using a separate fee payer and when the authority is also the fee payer. fix(driftClient.ts): update payer assignment to use wallet.payer if available for better flexibility fix(txHandler.ts): update feePayer assignment to use wallet.payer if available for improved compatibility feat(whileValidTxSender.ts): add optional timeout parameter to WhileValidTxSender for enhanced control over transaction sending feat(wallet.ts): refactor Wallet class to allow optional payer and improve signing logic style(helpers.ts): format empty async methods for consistency in MockUserMap class
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR is fixing the signing error with #1837 make fee payer function available. Please look at the original PR for the changes about fee-payer.
Additional changes
driftClientworks correctly.Walletinterface keeping backward compatibility