district0x / name-bazaar

A peer-to-peer marketplace for the exchange of names registered via the Ethereum Name Service
Eclipse Public License 1.0
85 stars 23 forks source link

Update ENS, openzeppelin and our contracts to Solidity 0.8 #199

Closed ladislavdubravsky closed 3 years ago

ladislavdubravsky commented 3 years ago

Summary

We update all contracts to Solidity 0.8 to be able to test dev setups with the coming https://github.com/ensdomains/name-wrapper and other ENS and openzeppelin updates without bothering with multiple compiler version setups.

Review notes

We had to mainly accommodate for: https://docs.soliditylang.org/en/v0.6.0/060-breaking-changes.html https://docs.soliditylang.org/en/v0.7.0/070-breaking-changes.html https://docs.soliditylang.org/en/breaking/080-breaking-changes.html

Some points:

Testing notes

For QA new contracts deploy to Ropsten will be needed.