Skip to content

feat(TeacherTools): Redesign Match summary to organize by choice#2299

Merged
breity merged 4 commits intodevelopfrom
match-summary-redesign
Apr 7, 2026
Merged

feat(TeacherTools): Redesign Match summary to organize by choice#2299
breity merged 4 commits intodevelopfrom
match-summary-redesign

Conversation

@breity
Copy link
Copy Markdown
Member

@breity breity commented Apr 6, 2026

Changes

  • Instead of showing each bucket, Match summaries now show each choice with the number of times the choice was moved into different buckets.
Screenshot 2026-04-06 at 11 15 13 AM

Test

  • In the Teacher Tools, ensure that Match summaries now list all the choices instead of buckets, with the number of times the choice was moved to different buckets.
  • Choices should be ordered by total number of times they were moved to any bucket, then alphabetically.
  • Choices that were not moved by any students to any buckets should say so and have lighter gray text.

@breity breity requested a review from hirokiterashima April 6, 2026 18:16
@breity breity self-assigned this Apr 6, 2026
@qltysh
Copy link
Copy Markdown

qltysh bot commented Apr 6, 2026

Qlty

Coverage Impact

Unable to calculate total coverage change because base branch coverage was not found.

Modified Files with Diff Coverage (3)

RatingFile% DiffUncovered Line #s
New file Coverage rating: C
.../teacher-summary-display/summary-data/MatchSummaryDataPoint.ts66.7%11
New file Coverage rating: A
...tives/teacher-summary-display/summary-data/MatchSummaryData.ts93.3%63
New file Coverage rating: A
...splay/match-summary-display/match-summary-display.component.ts77.8%50-51
Total85.2%
🤖 Increase coverage with AI coding...

In the `match-summary-redesign` branch, add test coverage for this new code:

- `src/assets/wise5/directives/teacher-summary-display/match-summary-display/match-summary-display.component.ts` -- Line 50-51
- `src/assets/wise5/directives/teacher-summary-display/summary-data/MatchSummaryData.ts` -- Line 63
- `src/assets/wise5/directives/teacher-summary-display/summary-data/MatchSummaryDataPoint.ts` -- Line 11

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

Copy link
Copy Markdown
Member

@hirokiterashima hirokiterashima left a comment

Choose a reason for hiding this comment

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

LGTM

@breity breity merged commit 707b27a into develop Apr 7, 2026
1 check passed
@breity breity deleted the match-summary-redesign branch April 7, 2026 20:50
@hirokiterashima
Copy link
Copy Markdown
Member

🎉 This PR is included in version 5.224.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants