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

Remove redundant call to verifyTransfer in method of NFT module #9102

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

Incede
Copy link
Contributor

@Incede Incede commented Oct 18, 2023

What was the problem?

This PR resolves #9101

How was it solved?

Remove the check as specified

How was it tested?

All tests pass already

@Incede Incede requested review from shuse2 and bobanm October 18, 2023 13:47
@Incede Incede self-assigned this Oct 18, 2023
@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

Merging #9102 (ea2cc6b) into release/6.1.0 (9747cce) will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.1.0    #9102      +/-   ##
=================================================
- Coverage          84.34%   84.34%   -0.01%     
=================================================
  Files                652      652              
  Lines              23967    23966       -1     
  Branches            3489     3489              
=================================================
- Hits               20215    20214       -1     
  Misses              3752     3752              
Files Coverage Δ
framework/src/modules/nft/method.ts 98.37% <ø> (-0.01%) ⬇️

@shuse2 shuse2 merged commit 185e3fa into release/6.1.0 Oct 19, 2023
10 checks passed
@shuse2 shuse2 deleted the remove-redundant-check branch October 19, 2023 08:54
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