Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Change order of class storage to store after charge fee #765

Merged
merged 7 commits into from
Jul 5, 2023

Conversation

mmsc2
Copy link
Contributor

@mmsc2 mmsc2 commented Jul 4, 2023

TITLE

Description

Description of the pull request changes and motivation.
fixes ticket #762

@mmsc2 mmsc2 linked an issue Jul 4, 2023 that may be closed by this pull request
@codecov-commenter
Copy link

codecov-commenter commented Jul 4, 2023

Codecov Report

Merging #765 (6479e32) into main (156254d) will increase coverage by 0.00%.
The diff coverage is 92.30%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #765   +/-   ##
=======================================
  Coverage   92.49%   92.50%           
=======================================
  Files          50       50           
  Lines       11893    11896    +3     
=======================================
+ Hits        11001    11004    +3     
  Misses        892      892           
Impacted Files Coverage Δ
src/transaction/declare.rs 98.63% <91.66%> (+<0.01%) ⬆️
src/transaction/declare_v2.rs 86.18% <100.00%> (ø)

@matias-gonz matias-gonz enabled auto-merge July 5, 2023 13:56
@matias-gonz matias-gonz added this pull request to the merge queue Jul 5, 2023
Merged via the queue into main with commit 7d9a628 Jul 5, 2023
@juanbono juanbono deleted the FixDeclare branch July 31, 2023 23:37
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.

Save declared class only if the fee charge was successful
5 participants