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

Update recover method of NFT module to validate NFT attributes array #9000

Merged
merged 3 commits into from
Sep 20, 2023

Conversation

Incede
Copy link
Contributor

@Incede Incede commented Sep 18, 2023

What was the problem?

This PR resolves #8974

How was it solved?

Update as specified

How was it tested?

Added unit test

@Incede Incede requested review from shuse2 and has5aan September 18, 2023 07:25
@Incede Incede self-assigned this Sep 18, 2023
@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #9000 (36f167e) into release/6.1.0 (a869773) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

❗ Current head 36f167e differs from pull request most recent head 7fc31fc. Consider uploading reports for the commit 7fc31fc to get more accurate results

Impacted file tree graph

@@                Coverage Diff                @@
##           release/6.1.0    #9000      +/-   ##
=================================================
- Coverage          84.36%   84.36%   -0.01%     
=================================================
  Files                651      650       -1     
  Lines              24013    24024      +11     
  Branches            3493     3490       -3     
=================================================
+ Hits               20258    20267       +9     
- Misses              3755     3757       +2     
Files Changed Coverage
framework/src/modules/nft/method.ts 100.00%

@shuse2 shuse2 enabled auto-merge (squash) September 20, 2023 09:33
@shuse2 shuse2 merged commit da62e35 into release/6.1.0 Sep 20, 2023
@shuse2 shuse2 deleted the 8974-validate branch September 20, 2023 18:02
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