`onERC721Received()` callback is never called when new tokens are minted in Erc721Facet.sol
The ERC721Facet contract does not properly call the corresponding callback when new tokens are minted. The ERC721 standard states that the onERC721Received callback must be called when a mint or transfer operation occurs. However, the smart contracts interacting as users with `Erc721Facet.mintNFT()` will not be notified with the onERC721Received callback, as expected according to the ERC721 standard.\\n`onErc721Received()` isn't called on minting:\\n```\\n    function mintNFT(address asset, uint8 shortRecordId)\\n        external\\n        isNotFrozen(asset)\\n        nonReentrant\\n        onlyValidShortRecord(asset, msg.sender, shortRecordId)\\n    {\\n        if (shortRecordId == Constants.SHORT_MAX_ID) {\\n            revert Errors.CannotMintLastShortRecord();\\n        }\\n        STypes.ShortRecord storage short =\\n            s.shortRecords[asset][msg.sender][shortRecordId];\\n\\n        if (short.tokenId != 0) revert Errors.AlreadyMinted();\\n\\n        s.nftMapping[s.tokenIdCounter] = STypes.NFT({\\n            owner: msg.sender,\\n            assetId: s.asset[asset].assetId,\\n            shortRecordId: shortRecordId\\n        });\\n\\n        short.tokenId = s.tokenIdCounter;\\n\\n        //@dev never decreases\\n        s.tokenIdCounter += 1;\\n    }\\n```\\n
Call `onErc721Received()`
It can create interoperability issues with users' contracts
```\\n    function mintNFT(address asset, uint8 shortRecordId)\\n        external\\n        isNotFrozen(asset)\\n        nonReentrant\\n        onlyValidShortRecord(asset, msg.sender, shortRecordId)\\n    {\\n        if (shortRecordId == Constants.SHORT_MAX_ID) {\\n            revert Errors.CannotMintLastShortRecord();\\n        }\\n        STypes.ShortRecord storage short =\\n            s.shortRecords[asset][msg.sender][shortRecordId];\\n\\n        if (short.tokenId != 0) revert Errors.AlreadyMinted();\\n\\n        s.nftMapping[s.tokenIdCounter] = STypes.NFT({\\n            owner: msg.sender,\\n            assetId: s.asset[asset].assetId,\\n            shortRecordId: shortRecordId\\n        });\\n\\n        short.tokenId = s.tokenIdCounter;\\n\\n        //@dev never decreases\\n        s.tokenIdCounter += 1;\\n    }\\n```\\n