Skip to content

Conversation

harry-rhesis
Copy link
Contributor

This PR introduces changes from the fix/metrics-inconsistencies branch.

📝 Summary

📁 Files Changed ( 9 files)

apps/backend/src/rhesis/backend/app/schemas/metric.py
apps/backend/uv.lock
apps/frontend/src/app/(protected)/metrics/[identifier]/page.tsx
apps/frontend/src/app/(protected)/metrics/components/MetricsClient.tsx
apps/frontend/src/app/(protected)/metrics/components/MetricsDirectoryTab.tsx
apps/frontend/src/app/(protected)/metrics/new/page.tsx
apps/frontend/src/components/common/BaseWorkflowSection.tsx
apps/frontend/src/components/common/NotificationContext.tsx
apps/frontend/src/utils/api-client/interfaces/metric.ts

📋 Commit Details

72abc51 - fix: metrics edit, clear visual feedback when the save operation is in progress (Rhesis Engineering, 2025-09-03 17:28)
8e76427 - fix: Page re-renders when typing in the edit text field (Rhesis Engineering, 2025-09-03 17:24)
5c90d3d - Fixing mutliple API calls in metric edit (Rhesis Engineering, 2025-09-03 17:03)
ed102ab - fix: clear feedback to users when they click 'Create Metric' (Rhesis Engineering, 2025-09-03 14:54)
12f733a - fix: Fixing display of metrics confirmation page during creation (Rhesis Engineering, 2025-09-03 10:58)
d698405 - ensuring that each TextField component has a stable reference (Rhesis Engineering, 2025-09-02 20:48)
7d01083 - fix: Making name an optional field for metric edits (Rhesis Engineering, 2025-09-02 20:44)
298be65 - fix: Adding individual loading states for the two tabs (Rhesis Engineering, 2025-09-02 20:29)
4311f0f - fix: Redirecting user to metrics directory after metrics creation (Rhesis Engineering, 2025-09-02 20:09)
42910a9 - fix: optimizing number of calls for creating custom metric (Rhesis Engineering, 2025-09-02 20:07)
20cd390 - fix: adding spinner for metrics deletion (Rhesis Engineering, 2025-09-02 20:00)
3e80686 - fix: Now displaying metrics not associated with behaviors (Rhesis Engineering, 2025-09-02 19:57)

✅ Checklist

  • Code follows the project's style guidelines
  • Self-review of code has been performed
  • Code is commented, particularly in hard-to-understand areas
  • Corresponding changes to documentation have been made
  • Tests have been added/updated for new functionality
  • All tests pass locally

🧪 Testing

📸 Screenshots (if applicable)

🔗 Related Issues

@harry-rhesis harry-rhesis self-assigned this Sep 3, 2025
@harry-rhesis harry-rhesis merged commit eda3f88 into main Sep 3, 2025
7 checks passed
@harry-rhesis harry-rhesis deleted the fix/metrics-inconsistencies branch September 3, 2025 15:30
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