Skip to content
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

Integrate Seen Tx Checkpoint API #875

Merged
merged 1 commit into from
Jan 15, 2025
Merged

Integrate Seen Tx Checkpoint API #875

merged 1 commit into from
Jan 15, 2025

Conversation

samholmes
Copy link
Collaborator

@samholmes samholmes commented Jan 2, 2025

  • WIP Upgrade edge-core-js local
  • Integrate new Seen Tx Checkpoint API

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

  • Yes
  • No

Dependencies

none

Description

none

@samholmes samholmes marked this pull request as draft January 2, 2025 21:02
@samholmes samholmes force-pushed the sam/seenTxCheckpoint branch from 84ce95f to aaf2474 Compare January 2, 2025 21:44
Copy link
Contributor

@swansontec swansontec left a comment

Choose a reason for hiding this comment

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

Approving, since the changes are tiny.

src/cardano/CardanoEngine.ts Outdated Show resolved Hide resolved
src/cardano/CardanoEngine.ts Outdated Show resolved Hide resolved
src/polkadot/PolkadotEngine.ts Outdated Show resolved Hide resolved
@samholmes samholmes force-pushed the sam/seenTxCheckpoint branch 4 times, most recently from 0e6bfcc to 6e0856a Compare January 15, 2025 00:36
Copy link
Contributor

@swansontec swansontec left a comment

Choose a reason for hiding this comment

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

This looks good, but I found a cleanup that might be worth doing since we're here.

@@ -387,13 +386,13 @@ export class TezosEngine extends CurrencyEngine<
// eslint-disable-next-line @typescript-eslint/no-floating-promises
this.addToLoop('checkTransactionsInnerLoop', TRANSACTION_POLL_MILLISECONDS)
// eslint-disable-next-line @typescript-eslint/no-floating-promises
super.startEngine(opts)
super.startEngine()
Copy link
Contributor

Choose a reason for hiding this comment

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

This could be await super.startEngine(), and then we can delete the comment above.

@samholmes samholmes force-pushed the sam/seenTxCheckpoint branch from 6e0856a to 53b7ee9 Compare January 15, 2025 06:17
@samholmes samholmes force-pushed the sam/seenTxCheckpoint branch from 53b7ee9 to 5224e56 Compare January 15, 2025 23:11
@samholmes samholmes marked this pull request as ready for review January 15, 2025 23:11
@samholmes samholmes enabled auto-merge January 15, 2025 23:11
@samholmes samholmes merged commit 251116c into master Jan 15, 2025
2 checks passed
@samholmes samholmes deleted the sam/seenTxCheckpoint branch January 15, 2025 23:17
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