First-reading review and ANU change diffs - #147
Closed
HarryRandall wants to merge 7 commits into
Closed
HarryRandall wants to merge 7 commits into
HarryRandall wants to merge 7 commits into
Conversation
|
Deployment failed for project coursemap with the following error: Learn More: https://vercel.com/coursemap?upgradeToPro=build-rate-limit |
This was referenced Sep 26, 2026
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
1 task
Owner
Author
|
Replaced by #153 (same commits, branch renamed to Generated by Claude Code |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requested by haz · project thread
Summary
Merge after #146.
Before: a record's first ANU reading could be published unreviewed, and changes showed as raw values.
After: a record's first reading must be reviewed before publishing, every field is rated with a confidence, and fields can be sent back for review from All fields. ANU changes show as a git-style diff with the model's notes on the change they concern, and values read as tables or requisite graphs. Record actions move into a menu beside the title, a stuck sync can be stopped, a failed sync can be retried from its alert or diagnostics, and the sync run page is simpler.
How: four feature commits (first-reading review and confidence with migration 016, diffs and notes, record actions menu, sync retry and run page), then test updates: unrated readings are taken as read, and the browser tests use the record actions menu and find the record name in the breadcrumb on non-Content tabs.
Verification
tsc, unit, database and browser tests pass; CI greenDeployment notes
Migration
016_first_read_review.sql. It is applied to haz's local Supabase only. Merging tomainruns the production migration job.