Skip to content

Refactor heartbeat to also record last query time#194

Merged
mboynes merged 8 commits intomainfrom
hotfix/stale-heartbeat-fixes
May 17, 2025
Merged

Refactor heartbeat to also record last query time#194
mboynes merged 8 commits intomainfrom
hotfix/stale-heartbeat-fixes

Conversation

@mboynes
Copy link
Copy Markdown
Contributor

@mboynes mboynes commented May 15, 2025

  • POSSIBLE BREAKING CHANGE Refactors SP_Heartbeat to store the last time the beat was checked as well as the last time the beat was verified. SP_Heartbeat::get_last_beat() previously returned an int, but now returns an array.
  • Includes assorted improvements for IDE static analysis.

Resolves #61, #62

jakewrfoster
jakewrfoster previously approved these changes May 16, 2025
Copy link
Copy Markdown
Member

@jakewrfoster jakewrfoster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

little suggestions, but nothing blocking
🍣


### 0.5.2

* **POSSIBLE BREAKING CHANGE** Refactors `SP_Heartbeat` to store the last time the beat was checked as well as the last time the beat was verified. `SP_Heartbeat::get_last_beat()` previously returned an int, but now returns an array.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm assuming the possible work around here would be to manually delete this option value?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not following -- do you mean what a plugin/theme currently using SP_Heartbeat::get_last_beat() should do to work around the change?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct!

Co-authored-by: Jake  <jake@alley.co>
@mboynes mboynes merged commit f626a93 into main May 17, 2025
5 checks passed
@mboynes mboynes deleted the hotfix/stale-heartbeat-fixes branch May 17, 2025 20:26
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.

Heartbeat should timestamp its last check

2 participants