aureus/watchcraft
1
0
Fork 0
Code Issues Pull requests Projects Releases 5 Packages 1 Wiki Activity Actions

feature/ratings-management-page #7

Merged
aureus merged 4 commits from feature/ratings-management-page into main 2026-05-25 15:29:19 +00:00
Conversation 0 Commits 4 Files changed 11 +478 -7
aureus commented 2026-05-25 15:06:22 +00:00
Owner
Copy link
No description provided. 
Adds a two-section settings page where users can review their rated items
(with an inline re-rate widget after clearing) and remove items from their
Not Interested list to re-enter the discovery queue.

No schema changes — all data is already in ratings, user_discovery,
library, and media_metadata.
test(browser): add Playwright tests for ratings management page
Some checks failed
PR Checks / test (pull_request) Failing after 2m23s
Details
6909568055 
6 structural tests in settings-ratings.test.ts: nav link visible in
dropdown, clicking it navigates to /settings/ratings, page loads without
JS errors, both section headings present, empty state message visible.

3 mutation tests in z-settings-ratings.test.ts (z- runs last): self-seeds
by rating a curate card, then verifies the item appears on the ratings
page, Clear button swaps the row to the re-rate widget, and Dismiss removes
the widget.

Snapshot for the empty-state page added to snapshots.test.ts.
aureus force-pushed feature/ratings-management-page from 6909568055
Some checks failed
PR Checks / test (pull_request) Failing after 2m23s
Details
to f474c70f16
Some checks failed
PR Checks / test (pull_request) Has been cancelled
Details
2026-05-25 15:12:26 +00:00 Compare
fix(tests): use select-after-insert for rating ID in PG integration tests
Some checks failed
PR Checks / test (pull_request) Failing after 3m10s
Details
2fb5fb61b9 
Knex insert() returns the inserted row count in Postgres, not the ID —
destructuring [ratingId] from the result was always undefined. Follow
the existing pattern in this test file: insert, then select by known
unique fields to get the id.
fix(tests): pass Date objects for datetime columns in MariaDB tests
All checks were successful
PR Checks / test (pull_request) Successful in 3m51s
Details
6934dd0f95 
MariaDB rejects ISO 8601 strings in DATETIME columns. Pass new Date()
directly so Knex formats the value correctly for each DB backend.
aureus merged commit d39ad749e9 into main 2026-05-25 15:29:19 +00:00
aureus referenced this pull request from a commit 2026-05-25 15:29:21 +00:00
feature/ratings-management-page (#7)
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aureus/watchcraft!7
No description provided.