Skip to content

Fix reminder list endpoint to include disabled reminders#1386

Merged
Aaronontheweb merged 1 commit into
netclaw-dev:devfrom
d0pare:fix/include-disabled-reminders-list
Jun 10, 2026
Merged

Fix reminder list endpoint to include disabled reminders#1386
Aaronontheweb merged 1 commit into
netclaw-dev:devfrom
d0pare:fix/include-disabled-reminders-list

Conversation

@d0pare

@d0pare d0pare commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update GET /api/reminders to request all reminder definitions from the reminder manager instead of active-only reminders.
  • Update the endpoint summary from “List all active reminders” to “List all reminders.”

Behavior

Disabled reminders remain persisted and can be re-enabled by ID, so the REST list endpoint now returns them in the default list response. This keeps the CLI-backed list output useful after a reminder has been disabled but not permanently deleted.

Validation

  • Not run; change is limited to the reminder list endpoint request flag and OpenAPI summary text.

@Aaronontheweb Aaronontheweb left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Aaronontheweb Aaronontheweb enabled auto-merge (squash) June 10, 2026 15:11
@Aaronontheweb Aaronontheweb merged commit f0b9671 into netclaw-dev:dev Jun 10, 2026
15 checks passed
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.

2 participants