Skip to content

chore: separate dashboard slider from dashboard provider + refactor#10572

Open
AshwinBhatkal wants to merge 2 commits intomainfrom
chore/issue-4053-remove-dashboard-slider-dashboard-provider
Open

chore: separate dashboard slider from dashboard provider + refactor#10572
AshwinBhatkal wants to merge 2 commits intomainfrom
chore/issue-4053-remove-dashboard-slider-dashboard-provider

Conversation

@AshwinBhatkal
Copy link
Contributor

@AshwinBhatkal AshwinBhatkal commented Mar 12, 2026

📄 Summary

  • Removes dashboard slider state from dashboard provider
  • Renamed to what it actually means - panel type selection modal
  • Refactor of some constants

Screenshots / Screen Recordings

Screen.Recording.2026-03-12.at.7.20.18.PM.mov
Screen.Recording.2026-03-12.at.7.22.46.PM.mov

Issues closed by this PR

Reference issues using Closes #issue-number to enable automatic closure on merge.

Closes https://github.com/SigNoz/engineering-pod/issues/4053

Part of https://github.com/SigNoz/engineering-pod/issues/3953

@github-actions github-actions bot added the chore label Mar 12, 2026
@@ -0,0 +1,97 @@
import { memo } from 'react';
Copy link
Contributor Author

@AshwinBhatkal AshwinBhatkal Mar 12, 2026

Choose a reason for hiding this comment

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

existing component

  1. Renamed
  2. memoized
  3. removal of useDashboard hook
  4. unification of queryParams

@AshwinBhatkal AshwinBhatkal marked this pull request as ready for review March 12, 2026 13:54
@AshwinBhatkal AshwinBhatkal requested review from a team as code owners March 12, 2026 13:54
},
};

export const PANEL_TYPES_INITIAL_QUERY: Record<PANEL_TYPES, Query> = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

has been moved from the constants file with a change in line 573

@@ -1,5 +1,6 @@
// ** Helpers
Copy link
Contributor Author

Choose a reason for hiding this comment

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

copied over from constants file except for 1 change

@AshwinBhatkal AshwinBhatkal force-pushed the chore/issue-4053-remove-dashboard-slider-dashboard-provider branch from db19e8f to dd15db9 Compare March 12, 2026 15:19
@AshwinBhatkal AshwinBhatkal enabled auto-merge March 12, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant