Skip to content

Conversation

@mbuechse
Copy link
Contributor

resolves #951

resolves #951

Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
@mbuechse mbuechse requested a review from garloff January 23, 2026 14:57
Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
@mbuechse mbuechse marked this pull request as ready for review January 23, 2026 20:15
Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
this can happen as long as the version column is still in the db

Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
@mbuechse
Copy link
Contributor Author

@garloff @fkr @depressiveRobot
This is now ready for review. I have taken the liberty to deploy it to the server, so you can already see how it works (or doesn't).

Benefits:

  • Every single file touched here is becoming smaller. In total, we're losing almost 300 lines
  • We are also losing obscure concepts, such as selector expressions.
  • Everyone is going to save a click each time they want to jump from the details page into the report. The detour via uuid is gone at long last!
  • This is going to pave the way for future improvements, such as [Feature Request] Restructure CM details page #1055

Please give it a chance.

Signed-off-by: Matthias Büchse <matthias.buechse@alasca.cloud>
@mbuechse mbuechse merged commit 817d960 into main Jan 23, 2026
10 checks passed
@mbuechse mbuechse deleted the issue/951 branch January 23, 2026 23:01
Copy link
Member

@garloff garloff left a comment

Choose a reason for hiding this comment

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

Nice work 👍

@mbuechse
Copy link
Contributor Author

@garloff Thanks! It's one step in a long sequence of steps, I'm afraid, but an important one. Many things are still suboptimal, but I hope we can address them later.

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.

[Feature Request] Re-engineer testcases and results handling

4 participants