Skip to content

chore/fix-open-limit - #62

Merged
brendan-ch merged 11 commits into
mainfrom
chore/fix-open-limit
Aug 7, 2026
Merged

chore/fix-open-limit#62
brendan-ch merged 11 commits into
mainfrom
chore/fix-open-limit

Conversation

@brendan-ch

@brendan-ch brendan-ch commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Make several substantial changes to how the open limit works.

  • After an open limit shields, the app now applies a "just blocked" delay so that a user can't immediately use another open
    • More in line with the original vision of the open limit
  • Call stopMonitoring right before startMonitoring when granting the open
    • Fix a freeze that was happening when using an open
  • Assert that the granted open session has actually expired before re-applying the shield and expiring it
    • This is because intervalDidEnd can fire on activity monitor replacement
  • Refactor several parts of the code that were not readable

@brendan-ch brendan-ch self-assigned this Aug 7, 2026
@brendan-ch
brendan-ch merged commit 89ccf15 into main Aug 7, 2026
2 checks passed
@brendan-ch
brendan-ch deleted the chore/fix-open-limit branch August 7, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant