Refactor `NFTCollection` contract - Githubissues

smartcontractkit / chainlink-automation-templates

Real-world sample projects using Chainlink Automation

https://automation.chainlink-demo.app

35 stars 14 forks source link

Refactor `NFTCollection` contract #43

Closed imollov closed 2 years ago

imollov commented 2 years ago

Use OZ Strings library
Use OZ Base64 library
Use OZ IERC20
Restrict visibility of state that doesn't need to be public
Improve formatting of JSON and XML strings
Improve ordering and grouping of functions
Use s_ prefix for storage variables
Name immutable state as constants (SCREAMING_SNAKE_CASE)

imollov commented 2 years ago

resolved in #46