Skip to content

Changed renderer for GradedTaskGroupDto to number#1017

Merged
SzBeni2003 merged 2 commits intostagingfrom
fix/task-number-renderer
Mar 4, 2026
Merged

Changed renderer for GradedTaskGroupDto to number#1017
SzBeni2003 merged 2 commits intostagingfrom
fix/task-number-renderer

Conversation

@SzBeni2003
Copy link
Contributor

@SzBeni2003 SzBeni2003 commented Feb 18, 2026

Summary by CodeRabbit

  • Bug Fixes
    • Fixed display formatting for task grading status counts in evaluation summaries.

@vercel
Copy link

vercel bot commented Feb 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cmsch-golyakorte Ready Ready Preview, Comment Mar 4, 2026 8:41pm
cmsch-skktv Ready Ready Preview, Comment Mar 4, 2026 8:41pm

Request Review

@SzBeni2003 SzBeni2003 merged commit d74c1c9 into staging Mar 4, 2026
3 of 9 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Mar 4, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a58a23ed-fcbb-487f-996a-12c1575869ee

📥 Commits

Reviewing files that changed from the base of the PR and between 9e75bb4 and c9dca59.

📒 Files selected for processing (1)
  • backend/src/main/kotlin/hu/bme/sch/cmsch/component/task/GradedTaskGroupDto.kt

📝 Walkthrough

Walkthrough

Three properties in a Kotlin DTO class received updated @property:GenerateOverview annotations with an explicit renderer parameter (OverviewType.NUMBER) added to specify numeric rendering behavior.

Changes

Cohort / File(s) Summary
Annotation Metadata Updates
backend/src/main/kotlin/hu/bme/sch/cmsch/component/task/GradedTaskGroupDto.kt
Added renderer = OverviewType.NUMBER parameter to @property:GenerateOverview annotations on three numeric properties (approved, rejected, notGraded) to explicitly specify rendering behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three little numbers dressed up fine,
With renderers specified on each line,
No logic bent, no flow rearranged,
Just annotations gently changed! ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/task-number-renderer

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

2 participants