Skip to content

Conversation

d4mr
Copy link
Member

@d4mr d4mr commented Aug 30, 2024

PR-Codex overview

The focus of this PR is to enhance transaction retry functionality by adding a new route to retry failed transactions.

Detailed summary

  • Added a new route retryFailedTransaction to retry failed transactions
  • Updated ErroredTransaction type to include new transaction types
  • Modified sendTransactionWorker to update transaction status and add manuallyResentAt date
  • Added necessary imports and schemas for the new route

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@d4mr d4mr requested a review from arcoraven August 30, 2024 18:35
Reply: Static<typeof responseBodySchema>;
}>({
method: "POST",
url: "/transaction/retry",
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe rename to retry-failed since we already have a /retry

Comment on lines 52 to 53
deprecated: true,
hide: true,
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove

@arcoraven arcoraven merged commit 9fd4d9c into main Aug 30, 2024
4 checks passed
@arcoraven arcoraven deleted the pb/retry-failed branch August 30, 2024 19:57
d4mr added a commit that referenced this pull request Sep 1, 2024
* feat: Add retryFailedTransaction route to transaction API

* fix: Update retryFailedTransaction route URL to /transaction/retry-failed

* feat: Refactor retryFailedTransaction route to improve error handling

* feat: Register retryFailedTransaction route in transaction API

* allow retrying queued -> failed transactions
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