fix: weekday math

ArbitrumFoundation · Aug 29, 2024 · e3582d8 · e3582d8
1 parent fd2a46d
commit e3582d8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/gov-action-contracts/util/OfficeHoursAction.sol b/src/gov-action-contracts/util/OfficeHoursAction.sol
@@ -63,8 +63,8 @@ contract OfficeHoursAction {
         if (block.timestamp < minimumTimestamp) revert MinimumTimestampNotMet();
 
         // Convert timestamp to weekday (1 = Monday, 7 = Sunday)
-        // Addiding 4 because Unix epoch (January 1, 1970) was a Thursday
-        uint256 weekday = ((block.timestamp / 86_400 + 4) % 7);
+        // Addiding 3 because Unix epoch (January 1, 1970) was a Thursday
+        uint256 weekday = ((block.timestamp / 86_400 + 3) % 7 + 1);
         if (weekday < minDayOfWeek || weekday > maxDayOfWeek) revert OutsideOfficeDays();
 
         // This is UTC time, leap seconds are not accounted for