@Vectorized good catch, the logic here was a bit wonky. Was trying to ensure that accounts deployed on chains other than the NFT's home chain will still return a null owner post-hardfork. This can be accomplished in a simpler way by checking against deploymentChainId instead of the live block.chainid.
@Vectorized good catch, the logic here was a bit wonky. Was trying to ensure that accounts deployed on chains other than the NFT's home chain will still return a null owner post-hardfork. This can be accomplished in a simpler way by checking against
deploymentChainId
instead of the liveblock.chainid
.Added a test case to show the expected behavior.