Closed andreivladbrg closed 4 months ago
Also, tests for
Adminable
contract are missing. Is that intentional?
pretty much yeah, because the code is the exact code from lockup
contracts
I also found that precompiles and artifacts were not updated with NFT descriptor
you are correct, i've missed this, thanks
pushed a new commit to address the feedback, lmk if it looks good
Linked issues
Changelog
SablierNFTDescriptor
tokenURI
function inSablierFlowState
Adminable
contractAdminable
inSablierFlowState
nftDescriptor
state variablesetNFTDescriptor
function inSablierFlowState
contract