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

Update removeSupportAllNFTs method of NFT module #8981

Closed
Tracked by #7245
Incede opened this issue Sep 14, 2023 · 0 comments
Closed
Tracked by #7245

Update removeSupportAllNFTs method of NFT module #8981

Incede opened this issue Sep 14, 2023 · 0 comments
Assignees
Milestone

Comments

@Incede
Copy link
Contributor

Incede commented Sep 14, 2023

Description

The removeSupportAllNFTs method of the NFT module may not remove support of all NFTs if the ALL_SUPPORTED_NFTS_KEY is present. The getAll function will never return the ALL_SUPPORTED_NFTS_KEY key because it iterates over all keys of size LENGTH_CHAIN_ID but ALL_SUPPORTED_NFTS_KEY is the empty string.

Acceptance Criteria

  • In the removeSupportAllNFTs function, also remove the ALL_SUPPORTED_NFTS_KEY key
  • Add unit test

Additional Information

release/6.1.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants