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

Only save contracts if tx was successful #290

Merged
merged 1 commit into from
Oct 1, 2024
Merged

Only save contracts if tx was successful #290

merged 1 commit into from
Oct 1, 2024

Conversation

Endogen
Copy link
Contributor

@Endogen Endogen commented Sep 30, 2024

Description

Only save contracts if tx was successful

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I have performed a self-review of my own code
  • I have tested this change in my development environment.
  • I have added tests to prove that this change works
  • All existing tests pass after this change
  • I have added / updated documentation related to this change

@Endogen Endogen self-assigned this Sep 30, 2024
@Endogen Endogen added the bug Something isn't working label Sep 30, 2024
@Endogen Endogen merged commit 04cb9a6 into master Oct 1, 2024
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@crosschainer crosschainer crosschainer approved these changes

@duelingbenjos duelingbenjos Awaiting requested review from duelingbenjos

Assignees

@Endogen Endogen

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Endogen @crosschainer