Skip to content

Conversation

@vishvamsinh28
Copy link
Contributor

This PR refactors the MonitorATMEmptyState component from a class-based implementation to a functional component using React functional syntax.

Closes #2789

@vishvamsinh28 vishvamsinh28 requested a review from a team as a code owner May 19, 2025 18:40
@vishvamsinh28 vishvamsinh28 requested review from jkowall and removed request for a team May 19, 2025 18:40
@codecov
Copy link

codecov bot commented May 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.85%. Comparing base (8b3fe77) to head (f2ec809).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2790      +/-   ##
==========================================
- Coverage   96.85%   96.85%   -0.01%     
==========================================
  Files         256      256              
  Lines        7951     7947       -4     
  Branches     2079     2077       -2     
==========================================
- Hits         7701     7697       -4     
- Misses        249      250       +1     
+ Partials        1        0       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Vishvamsinh Vaghela <[email protected]>
@yurishkuro yurishkuro added the changelog:bugfix-or-minor-feature 🐞 Bug fixes, Minor Improvements label May 20, 2025
@yurishkuro yurishkuro enabled auto-merge May 20, 2025 21:39
@yurishkuro yurishkuro added this pull request to the merge queue May 20, 2025
Merged via the queue into jaegertracing:main with commit 041f1d9 May 20, 2025
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:bugfix-or-minor-feature 🐞 Bug fixes, Minor Improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate MonitorATMEmptyState to a functional React component

3 participants