search

graphprotocol / block-oracle

MIT License
7 stars 13 forks source link

feat: add ownership check and init #118

Closed juanmardefago closed 2 years ago

juanmardefago commented 2 years ago

Added the ownership check Will require some changes to the overmind setup to trigger changes in the config files that have the values of the initial owner

tilacog commented 2 years ago

@neysofu, this PR introduces new message types. Do you believe it would break something on the oracle side?

I'm running locally with hardhat for +10mins, no errors so far.

juanmardefago commented 2 years ago

@neysofu, this PR introduces new message types. Do you believe it would break something on the oracle side?

I'm running locally with hardhat for +10mins, no errors so far.

There shouldn't be an issue unless you were relying on message 4 being different than the change ownership, given that it doesn't modify the encoded values for previously existing messages, it's just an extension to the old encoding