Closed eliseoci closed 1 year ago
Thanks for the great feedback @larry0x! I've addressed your comments. Lmk if it looks good to you.
On another topic, I've mistakenly opened the PR from my fork's main branch instead of the feature one. I can open a new PR and close this one to not have those annoying merge pull request
commits in the log history, if you prefer.
I can open a new PR and close this one to not have those annoying
merge pull request
commits
I would just squash-merge
This PR includes a contract for implementing non-transferable (aka soulbound) NFTs.