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.

Sign up for GitHub

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

Jump to bottom

[BCI-3813] Use the transactionID as the idempotency key in the EVM chain writer #13829

Merged
merged 6 commits into from
Jul 12, 2024
Merged

[BCI-3813] Use the transactionID as the idempotency key in the EVM chain writer #13829

merged 6 commits into from
Jul 12, 2024

Conversation

nickcorin
Copy link
Contributor

  • evm: Use the transactionId as the idempotency key in the chainwriter
  • changeset: Add changeset tag

Requires Dependencies

Resolves Dependencies

@nickcorin nickcorin requested a review from a team as a code owner July 11, 2024 17:25
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@cedric-cordenier cedric-cordenier added this pull request to the merge queue Jul 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 12, 2024
@cedric-cordenier cedric-cordenier added this pull request to the merge queue Jul 12, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 12, 2024
@ilija42 ilija42 added this pull request to the merge queue Jul 12, 2024
Merged via the queue into develop with commit 51225f8 Jul 12, 2024
115 checks passed
@ilija42 ilija42 deleted the fix/use-txid-cw branch July 12, 2024 13:05
This was referenced Jul 12, 2024
Draft
Closed
@chainchad chainchad mentioned this pull request Jul 29, 2024
Merged
@github-actions github-actions bot mentioned this pull request Jul 31, 2024
Draft
@github-actions github-actions bot mentioned this pull request Aug 1, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amit-momin amit-momin amit-momin approved these changes

@ilija42 ilija42 ilija42 approved these changes

@silaslenihan silaslenihan silaslenihan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nickcorin @silaslenihan @ilija42 @amit-momin