Skip to content

feat(gradebook): import did-you-mean, conflict change-matrix, loading cues, and copy polish#8460

Closed
LWS49 wants to merge 1 commit into
lws49/ext-pr4a-import-correctnessfrom
lws49/ext-pr4b-import-ux
Closed

feat(gradebook): import did-you-mean, conflict change-matrix, loading cues, and copy polish#8460
LWS49 wants to merge 1 commit into
lws49/ext-pr4a-import-correctnessfrom
lws49/ext-pr4b-import-ux

Conversation

@LWS49

@LWS49 LWS49 commented Jun 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

Wizard UX for the external grade importer. The Verify step shows header mismatches with did-you-mean suggestions and a malformed-cell list, and identifies rows by their External ID with a resolved-name column for match confirmation. The conflict prompt is redesigned as a per-student matrix showing each changed cell old → new, dropping unchanged rows. Adds loading affordances, scroll handling for wide previews, typography fixes, a reassigned-identifier advisory, and copy polish throughout.

Design decisions

  • Conflicts are shown as a per-student change matrix with unchanged cells dropped - the previous flat list re-counted cells whose value was identical at two decimals; the matrix shows only real changes (old → new) grouped by the student they affect.
  • Identifier-first display (External ID with a resolved-name column) replaces name-only rows - under an identifier-mode header, showing only the resolved name read as a mismatch; pairing the CSV value with the resolved name confirms the match.

Regression prevention

Covers: the import wizard (template/roster/from-existing, out-of-range verify, did-you-mean, loading), and the conflict change-matrix table (struck-old/arrow/bold-new, em-dash for null sides and missing cells, new-fill vs existing fallback, header/row identity) (frontend suites green). No backend logic change (that is PR4a). Import-flow specs run on push/CI.

Screenshots worth adding: the redesigned conflict change-matrix, and the Verify step with a did-you-mean suggestion.

@LWS49

LWS49 commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by #8465 (import correctness + UX consolidated into a single import PR).

@LWS49 LWS49 closed this Jun 28, 2026
@LWS49 LWS49 deleted the lws49/ext-pr4b-import-ux branch June 28, 2026 14:34
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.

1 participant