Skip to content

[FEATURE] [MER-5611] Select all students in the support tile bucket#6562

Draft
nicocirio wants to merge 2 commits intomasterfrom
MER-5611-select-all-students-button-includes-students-not-visible
Draft

[FEATURE] [MER-5611] Select all students in the support tile bucket#6562
nicocirio wants to merge 2 commits intomasterfrom
MER-5611-select-all-students-button-includes-students-not-visible

Conversation

@nicocirio
Copy link
Copy Markdown
Contributor

@nicocirio nicocirio commented May 6, 2026

Jira

Summary

  • update the Student Support tile master checkbox to select all students in the current bucket/filter result, not just the visible page
  • show Select all (# students) when the master checkbox is checked to clarify the full selection scope
  • keep selected recipients available for the email modal even when some selected students are not yet visible due to pagination
  • update support tile work-item docs to record the MER-5611 behavior change from the original MER-5252 scope

What Was Added

  • support tile state now tracks the full filtered student set so select-all can operate across pagination
  • the checked-state label now reflects the total selected scope for the current bucket/filter result
  • support tile tests now cover cross-page selection and email recipient population for non-visible students
  • support tile work-item docs now reflect the MER-5611 behavior change and preserve the prior MER-5252 scope as historical context

Validation

  • mix format
  • mix compile
  • yarn --cwd assets run format
  • yarn --cwd assets run deploy
  • mix test test/oli_web/components/delivery/instructor_dashboard/student_support_tile_test.exs test/oli_web/components/delivery/instructor_dashboard/student_support_email_modal_test.exs
  • python3 /Users/nicocirio/.local/share/harness/skills/validate/scripts/validate_work_item.py docs/exec-plans/current/epics/intelligent_dashboard/support_tile --check all

Video Evidence

Screen.Recording.2026-05-06.at.11.19.05.AM.mov

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

Note: risk rules not loaded — micromatch is not a function

Generated by 🚫 dangerJS against f5f21bd

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

AI Review — security

No issues found

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

AI Review — performance

No issues found

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 6, 2026

AI Review — elixir

No issues found

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