OwnableUpgradeable Declares State Variables Outside Diamond Storage

[mudgen]

The example code on the README file inherits OwnableUpgradeable.sol from OpenZeppelin. OwnableUpgradeable.sol declares state variables outside Diamond Storage which can lead to storage collision if any other contracts, or inherited contracts declare state variables outside Diamond Storage. The best practice for diamonds is to declare all state variables within AppStorage or Diamond Storage.

SafeOwnable.sol from solidsate-solidity could be used instead, or other implementation.