tact icon indicating copy to clipboard operation
tact copied to clipboard

Add renounceOwnership to OwnableTransferable

Open talkol opened this issue 2 years ago • 3 comments

It's a good idea to standardize what it means to transfer ownership to nobody

Probably transfer to newAddress(0,0) - but we don't want users to think about these implementation details

talkol avatar Mar 11 '23 10:03 talkol

Also have a burner address to transfer assets to. I just not sure if this would be safe and won't give an easy way to make mistakes

ex3ndr avatar Mar 11 '23 11:03 ex3ndr

If ownership is indeed renounced and a user is going to an explorer to verify that, it will be nicer if the explorer prints out something standard instead of showing base64 encoding of the zero address which nobody will understand

Another reason to standardize the zero address as nobody

talkol avatar Mar 11 '23 22:03 talkol

I don't think adding renounceOwnership as a new message adds potential mistakes, It's not much different from sending by mistake your tokens to the wrong place or by accident transfer ownership to the wrong address

This is also the pattern on EVM:

https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/access/Ownable.sol

talkol avatar Mar 11 '23 22:03 talkol