Skip to content

chore: change label#7405

Merged
santicomp2014 merged 1 commit into
mainfrom
fix-assignment-type-labels-text
Jul 9, 2026
Merged

chore: change label#7405
santicomp2014 merged 1 commit into
mainfrom
fix-assignment-type-labels-text

Conversation

@gmorador-tribu

@gmorador-tribu gmorador-tribu commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

This pull request makes minor improvements to the assignment type labels in the AssignmentTypeSelector component, updating them for consistency and clarity.

  • Changed the label for the reading assignment type to "Social Annotation" and the label for the hide_and_reveal assignment type to "Paced Social Annotation" in AssignmentTypeSelector.tsx for improved clarity and consistency.

@santicomp2014
santicomp2014 merged commit 35a7c71 into main Jul 9, 2026
9 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates user-facing assignment-type labels in the frontend assignment type selector.

Changes:

  • Rename/capitalize the reading assignment label to “Social Annotation”.
  • Rename the hide_and_reveal assignment label to “Paced Social Annotation”.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 13 to 17
/** Human-readable label shown in the selector for each assignment type. */
const ASSIGNMENT_TYPE_LABELS: Record<AssignmentType, string> = {
reading: 'Social annotation',
hide_and_reveal: 'Guided Social annotation',
reading: 'Social Annotation',
hide_and_reveal: 'Paced Social Annotation',
};
Comment on lines +15 to +16
reading: 'Social Annotation',
hide_and_reveal: 'Paced Social Annotation',
@santicomp2014
santicomp2014 deleted the fix-assignment-type-labels-text branch July 9, 2026 18:43
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.

3 participants