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

Update fee after ccm execute logic #8601

Merged
merged 2 commits into from
Jun 20, 2023
Merged

Update fee after ccm execute logic #8601

merged 2 commits into from
Jun 20, 2023

Conversation

mitsuaki-u
Copy link
Contributor

What was the problem?

This PR resolves #8532

How was it solved?

Updated after ccm execute logic in fee module

How was it tested?

Updated relevant unit tests

@mitsuaki-u mitsuaki-u self-assigned this Jun 16, 2023
@codecov
Copy link

codecov bot commented Jun 16, 2023

Codecov Report

Merging #8601 (fd13291) into release/6.0.0 (89085a6) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff               @@
##           release/6.0.0    #8601   +/-   ##
==============================================
  Coverage          83.25%   83.25%           
==============================================
  Files                593      593           
  Lines              22241    22249    +8     
  Branches            3279     3283    +4     
==============================================
+ Hits               18516    18524    +8     
  Misses              3725     3725           
Impacted Files Coverage Δ
framework/src/modules/fee/cc_method.ts 100.00% <100.00%> (ø)
framework/src/modules/fee/module.ts 96.34% <100.00%> (+0.04%) ⬆️

Copy link
Contributor

@ishantiw ishantiw left a comment

Choose a reason for hiding this comment

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

LGTM

@ishantiw ishantiw merged commit 15c2ecf into release/6.0.0 Jun 20, 2023
10 checks passed
@ishantiw ishantiw deleted the 8532-ccm-burn branch June 20, 2023 14:04
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.

3 participants