[OPIK-4360] [FE] Implement view datasets permission check#5439
[OPIK-4360] [FE] Implement view datasets permission check#5439anastasiapyzhik merged 21 commits intomainfrom
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
...ntend/src/components/pages/OptimizationsPage/AddOptimizationDialog/AddOptimizationDialog.tsx
Show resolved
Hide resolved
apps/opik-frontend/src/components/pages-shared/experiments/useExperimentsGroupsAndFilters.ts
Outdated
Show resolved
Hide resolved
apps/opik-frontend/src/components/layout/DatasetsPageGuard/index.tsx
Outdated
Show resolved
Hide resolved
This comment was marked as resolved.
This comment was marked as resolved.
1 similar comment
📋 PR Linter Failed❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the |
This comment was marked as resolved.
This comment was marked as resolved.
apps/opik-frontend/src/components/pages/CompareExperimentsPage/CompareExperimentsPage.tsx
Outdated
Show resolved
Hide resolved
apps/opik-frontend/src/components/pages/CompareExperimentsPage/CompareExperimentsPage.tsx
Outdated
Show resolved
Hide resolved
This comment was marked as resolved.
This comment was marked as resolved.
1 similar comment
📋 PR Linter Failed❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the |
…d filtering" This reverts commit 0c2db57.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
1 similar comment
📋 PR Linter Failed❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the ❌ Missing Section. The description is missing the |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
|
🔄 Test environment deployment process has started Phase 1: Deploying base version You can monitor the progress here. |
|
✅ 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. |
Details
This PR implements comprehensive permission checks for the VIEW_DATASET permission across the frontend. Most notably, users without dataset viewing permissions can no longer access experiments at all, since experiments are fundamentally tied to datasets. This includes hiding the experiments page, sidebar navigation, and all experiment-related features. Additionally, dataset-specific functionality (optimizations, playground dataset selection, "Add to Dataset" actions) is also hidden or disabled based on permissions.
Key changes:
canViewDatasetspermission to the frontend permission system with cascading effects on experimentsDatasetsPageGuardcomponent to restrict access to the datasets pageScreenshots
Change checklist
Issues
Testing
Frontend Permission Checks:
Documentation