Skip to content

Conversation

Krosovok
Copy link
Contributor

@Krosovok Krosovok commented Aug 7, 2025

@Krosovok
Copy link
Contributor Author

Krosovok commented Aug 8, 2025

That's how it looks now. Showing the current status as well (as disabled and grayed-out). But You can't confirm the status unless you change it (and provide comment if needed). This way it is:

  1. Aligned with web UI;
  2. Gives user information what the current status is (because otherwise we don't provide this information and it is important);
  3. Provide better understanding of which statuses exist.
image

@Krosovok Krosovok marked this pull request as ready for review August 8, 2025 07:30
@Krosovok Krosovok requested a review from nquinquenel August 8, 2025 07:32
Copy link
Member

@nquinquenel nquinquenel left a comment

Choose a reason for hiding this comment

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

LGTM! Minor comments

@Krosovok Krosovok force-pushed the task/vt/SLI-2188-transition-options-alignment-with-web-UI branch from 773572b to ba7fe11 Compare August 8, 2025 10:08
Copy link

sonarqube-next bot commented Aug 8, 2025

@Krosovok Krosovok merged commit 44e12a2 into master Aug 8, 2025
29 checks passed
@Krosovok Krosovok deleted the task/vt/SLI-2188-transition-options-alignment-with-web-UI branch August 8, 2025 14:03
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