contract improvements suggestion

[thelostone-mc]

Description

This section documents feedback gotten from peeps who have glanced over the contracts + given suggestions.
Some of these might be

• taken on and changed made (with link to PR / commit)
• ignored (if so -> with reason to why it's not being done)

Suggested Improvements

• Ensure event are emitted when MetaPtr is set during init on RoundImplementation 9bbf53a
• Ensure event are emitted when MetaPtr is set during init on ProgramImplementation 9bbf53a
• Making RoundFactory.create accept encoded param (to give less reason to make changes in Factory) task: contract changes to Round contracts #139
• Making ProgramFactory.create accept encoded param (to give less reason to make changes in Factory) feat: program contract updates  #137
• Set multiple DEFAULT_ADMIN during create Round task: contract changes to Round contracts #139
• Set multiple DEFAULT_ADMIN during create Program feat: program contract updates  #137
• Set ROLE value in the initialize function (instead of hardcoding as constants) Reason: contract improvements suggestion #91 (comment) - @gravityrocks
• Make ProgramFactory contract upgradeable feat: factory contracts upgradable  #141
• Make RoundFactory contract upgradeable feat: factory contracts upgradable  #141

[thelostone-mc]

Set ROLE value in the initialize function (instead of hardcoding as constants) - @gravityrocks

We aren't implementing this ATM simply cause

• contracts are now upgradable
• cannot think of a reason why we would want to update the ROLE value (unless we are adding new roles -> which would inturn require a contract upgrade)
• additionally having it as a constant ensure it's there in the bytecode -> less gas costs -> aka cheaper by keeping it as it is