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

feat: implement transaction monitor #340

Merged
merged 6 commits into from
May 11, 2023
Merged

Conversation

mpetrun5
Copy link
Member

@mpetrun5 mpetrun5 commented May 3, 2023

Implement transaction monitor that sends transactions and resends them with higher gas if they are stuck.

Description

Related Issue Or Context

Closes: #338

How Has This Been Tested? Testing details.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation

Checklist:

  • I have commented my code, particularly in hard-to-understand areas.
  • I have ensured that all acceptance criteria (or expected behavior) from issue are met
  • I have updated the documentation locally and in chainbridge-docs.
  • I have added tests to cover my changes.
  • I have ensured that all the checks are passing and green, I've signed the CLA bot

@mpetrun5
Copy link
Member Author

mpetrun5 commented May 3, 2023

Still want to test somewhere there is London gas price because I suspect the tip can be 1 and after the increase it would still be 1 and it won't go through because it is the same.

@github-actions
Copy link

github-actions bot commented May 3, 2023

Go Test coverage is 63.7 %\ ✨ ✨ ✨

@github-actions
Copy link

github-actions bot commented May 8, 2023

Go Test coverage is 63.6 %\ ✨ ✨ ✨

@github-actions
Copy link

github-actions bot commented May 9, 2023

Go Test coverage is 63.7 %\ ✨ ✨ ✨

@mpetrun5 mpetrun5 requested a review from MakMuftic May 10, 2023 11:41
@github-actions
Copy link

Go Test coverage is 63.7 %\ ✨ ✨ ✨

Copy link
Member

@P1sar P1sar left a comment

Choose a reason for hiding this comment

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

In general looks good, left couple comments to check.

@mpetrun5 mpetrun5 requested a review from P1sar May 10, 2023 16:18
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ mpetrun5
❌ Matija Petrunic


Matija Petrunic seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions
Copy link

Go Test coverage is 63.7 %\ ✨ ✨ ✨

@mpetrun5 mpetrun5 merged commit 9ad5cad into main May 11, 2023
5 checks passed
@mpetrun5 mpetrun5 deleted the mpetrun5/transaction-monitor branch May 11, 2023 09:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Simple transaction queue
4 participants