Skip to content

feat(release-notes): add status filter to exclude Draft entries#29

Open
yecksin wants to merge 1 commit into
dev-release-notes-microservicefrom
claude/spanish-greeting-g6Z6D
Open

feat(release-notes): add status filter to exclude Draft entries#29
yecksin wants to merge 1 commit into
dev-release-notes-microservicefrom
claude/spanish-greeting-g6Z6D

Conversation

@yecksin
Copy link
Copy Markdown
Contributor

@yecksin yecksin commented Apr 10, 2026

Summary

  • Adds a status query param to the queryDatabase endpoint to filter release notes by their Notion "Status" property
  • By default (no status param), entries with Status=Draft are excluded — this means old release notes without a Status value remain visible without needing to update them
  • When status is explicitly provided (e.g. ?status=Draft), it filters to show only that specific status

Usage

GET /databases/:databaseId/query?projects=PRMS              → Excludes Draft (default)
GET /databases/:databaseId/query?projects=PRMS&status=Draft  → Only Draft
GET /databases/:databaseId/query?projects=PRMS&status=Published → Only Published

Notion setup required

Add a Select property called "Status" to the release notes database with options:

  • Draft (yellow) — for entries in progress
  • Published (green) — for entries ready to show

Entries without a Status value are treated as published (not filtered out).

Test plan

  • Unit tests updated for service and controller
  • 100% code coverage maintained
  • Verify in Notion that the "Status" Select property is created
  • Test API call without status param returns non-Draft entries
  • Test API call with ?status=Draft returns only Draft entries

https://claude.ai/code/session_01YXYTSDowYhjStxhVwGANPd

…m Notion query

By default, queryDatabase now excludes entries with Status=Draft so
published and untagged release notes remain visible without needing
to update old records. An optional `status` query param allows
explicitly filtering by any status value (e.g. ?status=Draft).

https://claude.ai/code/session_01YXYTSDowYhjStxhVwGANPd
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