-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New poly oracle #274
New poly oracle #274
Conversation
@@ -6,39 +6,57 @@ import "openzeppelin-solidity/contracts/ownership/Ownable.sol"; | |||
|
|||
contract MakerDAOOracle is IOracle, Ownable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamdossa since we removed all references to MakerDAO in the code, do you want to rename the contract too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Think it is OK to leave. This contract will only work w/ MakerDAO deployed Medianizer contracts.
contracts/oracles/MakerDAOOracle.sol
Outdated
* @param _medianizer Address of Maker medianizer | ||
*/ | ||
function changeMedianier(address _medianizer) public onlyOwner { | ||
emit LogChangeMedianizer(_medianizer, medianizer, now); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
require
_medianizer != address(0)?
@satyamakgec FYI - I changed to using an interface for the Medianizer which gets rid of some of the warnings. |
MakerDAOOracle can be reused for multiple oracles (Medianizer contracts)