Merge pull request #188 from overlay-market/s/set-heartbeat-event

Add `HeartbeatUpdated` event and emit it when heartbeat is updated
overlay-market · Mar 20, 2024 · 9d3f0ee · 9d3f0ee
2 parents c947791 + b440e00
commit 9d3f0ee
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/contracts/feeds/chainlink/OverlayV1ChainlinkFeed.sol b/contracts/feeds/chainlink/OverlayV1ChainlinkFeed.sol
@@ -12,6 +12,8 @@ contract OverlayV1ChainlinkFeed is OverlayV1Feed {
     uint256 public heartbeat;
     string public description;
 
+    event HeartbeatSet(uint256 heartbeat);
+
     modifier onlyGovernor() {
         require(ov.hasRole(GOVERNOR_ROLE, msg.sender), "OVV1: !governor");
         _;
@@ -29,7 +31,7 @@ contract OverlayV1ChainlinkFeed is OverlayV1Feed {
         aggregator = AggregatorV3Interface(_aggregator);
         decimals = aggregator.decimals();
         description = aggregator.description();
-        heartbeat = _heartbeat;
+        _setHeartbeat(_heartbeat);
         ov = IOverlayV1Token(_ov);
     }
 
@@ -123,6 +125,11 @@ contract OverlayV1ChainlinkFeed is OverlayV1Feed {
     }
 
     function setHeartbeat(uint256 _heartbeat) external onlyGovernor {
+        _setHeartbeat(_heartbeat);
+    }
+
+    function _setHeartbeat(uint256 _heartbeat) internal {
         heartbeat = _heartbeat;
+        emit HeartbeatSet(_heartbeat);
     }
 }