Skip to content

Conversation

karenska
Copy link
Member

@karenska karenska commented Oct 20, 2026 •

Added optional, case-insensitive artwork search by title and artist name. Search is applied before pagination, with matching totalCount and stable ordering. Empty searches return the full catalogue. Added five API tests covering search, case sensitivity, empty input, no matches and pagination. All tests, linting and TypeScript checks pass.

Closes #33

@karenska karenska added enhancement New feature or request test Changes related to tests labels Oct 20, 2026
@karenska karenska added this to the P2: #3 delivery milestone Oct 20, 2026
@karenska karenska requested a review from emmafre October 20, 2026 12:29
@karenska karenska self-assigned this Oct 20, 2026
@karenska karenska requested a review from a team as a code owner October 20, 2026 12:29
Copy link
Member

@emmafre emmafre left a comment

Choose a reason for hiding this comment

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

Looks good, and the tests pass! I assume the frontend search-component will be developed in a separate issue. If that's the case, this is ready to merge.

@karenska karenska merged commit be0971a into dev Oct 20, 2026
@karenska karenska deleted the 33-implement-artwork-search branch October 20, 2026 12:43
@emmafre emmafre removed enhancement New feature or request test Changes related to tests labels Oct 20, 2026
@emmafre emmafre linked an issue Oct 20, 2026 that may be closed by this pull request
7 tasks
@karenska karenska modified the milestone: P2: #3 delivery Oct 20, 2026
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement artwork search
2 participants