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

Remove unnecessary calls to validator.validate in commands of NFT module #9012

Merged
merged 2 commits into from
Sep 20, 2023

Conversation

Incede
Copy link
Contributor

@Incede Incede commented Sep 19, 2023

What was the problem?

This PR resolves #8987

How was it solved?

Update as specified

How was it tested?

Updated unit test

@Incede Incede self-assigned this Sep 19, 2023
@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Merging #9012 (4670a87) into release/6.1.0 (39b89af) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.1.0    #9012      +/-   ##
=================================================
- Coverage          84.36%   84.35%   -0.01%     
=================================================
  Files                650      650              
  Lines              24022    24017       -5     
  Branches            3490     3490              
=================================================
- Hits               20266    20260       -6     
- Misses              3756     3757       +1     
Files Changed Coverage
...amework/src/modules/nft/cc_commands/cc_transfer.ts ø
framework/src/modules/nft/commands/transfer.ts ø
...k/src/modules/nft/commands/transfer_cross_chain.ts ø

@shuse2 shuse2 merged commit 8b2339d into release/6.1.0 Sep 20, 2023
11 checks passed
@shuse2 shuse2 deleted the 8987-remove-validation branch September 20, 2023 13:58
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