Skip to content

Conversation

@janno42
Copy link
Member

@janno42 janno42 commented Jul 27, 2025

We didn't use single results for several years now. So let's remove lots of special handling!

This migrates all single results into "normal" evaluations with an overall rating questions holding the existing answers and removes all single results logic.

richardebeling
richardebeling previously approved these changes Jul 28, 2025
Copy link
Member

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

looks good, only a handful of small things

niklasmohrin
niklasmohrin previously approved these changes Jul 31, 2025
Copy link
Member

@niklasmohrin niklasmohrin left a comment

Choose a reason for hiding this comment

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

No more hot single results in this area 😎

@janno42 janno42 force-pushed the remove-single-results branch from c0ef65c to 5a22836 Compare August 11, 2025 13:39
@janno42 janno42 merged commit 6f52a6b into e-valuation:main Aug 11, 2025
15 checks passed
@janno42 janno42 deleted the remove-single-results branch August 11, 2025 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants