Skip to content

[OPIK-5657] [FE] fix: replace eval suite link with plain text on experiment page#6086

Merged
YarivHashaiComet merged 2 commits intomainfrom
yariv-h/OPIK-5657-remove-eval-suite-link-experiment
Apr 9, 2026
Merged

[OPIK-5657] [FE] fix: replace eval suite link with plain text on experiment page#6086
YarivHashaiComet merged 2 commits intomainfrom
yariv-h/OPIK-5657-remove-eval-suite-link-experiment

Conversation

@YarivHashaiComet
Copy link
Copy Markdown
Collaborator

Details

Replace the clickable evaluation suite link (NavigationTag) with a plain text Tag on the experiment page. Users were accidentally clicking on the eval suite link instead of the experiment. The evaluation suite name now displays as non-interactive text with the same visual style (yellow Database icon, bold text) but without navigation behavior.

  • Replaced NavigationTag with plain Tag for dataset name in both v1 and v2 CompareExperimentsDetails
  • Maintained visual consistency: yellow Database icon, comet-body-s-accented text weight
  • Other navigation tags (prompt, traces) remain clickable as before

Change checklist

  • User facing
  • Documentation update

Issues

  • OPIK-5657

AI-WATERMARK

AI-WATERMARK: yes

  • If yes:
    • Tools: Claude Code
    • Model(s): Claude Opus 4.6
    • Scope: Full implementation
    • Human verification: Code review, visual verification

Testing

  • cd apps/opik-frontend && npx eslint --cache — lint clean on changed files
  • Manual: Navigate to experiment page → verify eval suite name is plain text (not clickable) → verify prompt/traces tags still navigate correctly

Documentation

N/A

…riment page

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ataset is deleted

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@YarivHashaiComet YarivHashaiComet marked this pull request as ready for review April 6, 2026 14:26
@YarivHashaiComet YarivHashaiComet requested a review from a team as a code owner April 6, 2026 14:26
@YarivHashaiComet YarivHashaiComet added the test-environment Deploy Opik adhoc environment label Apr 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 6, 2026

🔄 Test environment deployment process has started

Phase 1: Deploying base version 1.10.59-4805 (from main branch) if environment doesn't exist
Phase 2: Building new images from PR branch yariv-h/OPIK-5657-remove-eval-suite-link-experiment
Phase 3: Will deploy newly built version after build completes

You can monitor the progress here.

@CometActions
Copy link
Copy Markdown
Collaborator

🌙 Nightly cleanup: The test environment for this PR (pr-6086) has been cleaned up to free cluster resources. PVCs are preserved — re-deploy to restore the environment.

@CometActions CometActions removed the test-environment Deploy Opik adhoc environment label Apr 7, 2026
@CometActions
Copy link
Copy Markdown
Collaborator

🌙 Nightly cleanup: The test environment for this PR (pr-6086) has been cleaned up to free cluster resources. PVCs are preserved — re-deploy to restore the environment.

1 similar comment
@CometActions
Copy link
Copy Markdown
Collaborator

🌙 Nightly cleanup: The test environment for this PR (pr-6086) has been cleaned up to free cluster resources. PVCs are preserved — re-deploy to restore the environment.

@YarivHashaiComet YarivHashaiComet added the test-environment Deploy Opik adhoc environment label Apr 9, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

🔄 Test environment deployment process has started

Phase 1: Deploying base version 1.10.59-4805 (from main branch) if environment doesn't exist
Phase 2: Building new images from PR branch yariv-h/OPIK-5657-remove-eval-suite-link-experiment
Phase 3: Will deploy newly built version after build completes

You can monitor the progress here.

@CometActions
Copy link
Copy Markdown
Collaborator

Test environment is now available!

To configure additional Environment variables for your environment, run [Deploy Opik AdHoc Environment workflow] (https://github.com/comet-ml/comet-deployment/actions/workflows/deploy_opik_adhoc_env.yaml)

Access Information

The deployment has completed successfully and the version has been verified.

Copy link
Copy Markdown
Collaborator

@aadereiko aadereiko left a comment

Choose a reason for hiding this comment

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

LGTM!

@YarivHashaiComet YarivHashaiComet merged commit 07a4f83 into main Apr 9, 2026
43 of 44 checks passed
@YarivHashaiComet YarivHashaiComet deleted the yariv-h/OPIK-5657-remove-eval-suite-link-experiment branch April 9, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Frontend test-environment Deploy Opik adhoc environment typescript *.ts *.tsx

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants