Skip to content

feat: time viewer for config changes#2885

Open
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-2355
Open

feat: time viewer for config changes#2885
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-2355

Conversation

Copy link
Contributor

Copilot AI commented Feb 24, 2026

Adds a time viewer component for config changes to visualize configuration history over time using a scatter plot graph.

Changes

  • ConfigChangesGraph.tsx: New scatter plot visualization component using reaviz library
  • ConfigChangesViewToggle.tsx: Toggle switch component to switch between Table and Graph views
  • ConfigDetailsChangesPage.tsx: Integration of the graph view with conditional rendering based on toggle state
  • ConfigRelatedChangesFilters.tsx: Added the view toggle to the filter bar

Status

⚠️ Blocked: This PR requires manual rebase to resolve conflicts with the current main branch. The codebase has evolved significantly and key components have changed APIs. A fresh implementation against current main may be needed.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Feb 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
aws-preview Ready Ready Preview Feb 24, 2026 8:05am
flanksource-ui Ready Ready Preview Feb 24, 2026 8:05am

Request Review

Copilot AI changed the title [WIP] Fix time viewer for configuration changes feat: time viewer for config changes Feb 24, 2026
Copilot AI requested a review from moshloop February 24, 2026 08:02
@moshloop moshloop marked this pull request as ready for review February 24, 2026 08:16
@moshloop moshloop changed the base branch from 93-Timeline-viewer to main February 24, 2026 08:16
@moshloop
Copy link
Member

@copilot perform a rebase and merge resolving conflicts

@moshloop
Copy link
Member

@copilot resolve the merge conflicts and open a new PR with a fresh commit

Copy link
Contributor Author

Copilot AI commented Feb 24, 2026

@copilot perform a rebase and merge resolving conflicts

I attempted to rebase this PR onto the latest main branch, but I cannot complete this operation due to several limitations:

  1. Cannot force push: My environment doesn't allow git push --force which is required after rebasing
  2. Significant API changes: The codebase has evolved significantly from v1.0.810 (this PR's base) to v1.4.192 (current main). Key components like Age, FilterByCellValue, MRTDataTable, and hooks like usePrefixedSearchParams have changed APIs that require manual resolution

Manual rebase steps needed:

git fetch origin main
git rebase origin/main
# Resolve conflicts, particularly in:
# - src/components/Configs/Changes/ConfigChangeTable.tsx
# - src/components/Configs/Changes/ConfigsRelatedChanges/FilterBar/ConfigRelatedChangesFilters.tsx  
# - src/pages/config/details/ConfigDetailsChangesPage.tsx
git push --force-with-lease

Alternatively, you could close this PR and I can create a fresh implementation of the time viewer feature against the current main branch.

Copy link
Contributor Author

Copilot AI commented Feb 24, 2026

@copilot resolve the merge conflicts and open a new PR with a fresh commit

I cannot open new PRs - this is outside my environment capabilities. I can only commit and push changes to this existing PR.

However, I can implement a fresh version of the time viewer feature against the current main branch and push those changes to this PR. Would you like me to proceed with that approach?

The implementation would:

  1. Create ConfigChangesGraph.tsx - scatter plot visualization using reaviz
  2. Create ConfigChangesViewToggle.tsx - toggle switch between Table/Graph views
  3. Update ConfigDetailsChangesPage.tsx to integrate the graph view
  4. Update ConfigRelatedChangesFilters.tsx to add the toggle
  5. Add the reaviz dependency to package.json

Let me know if you'd like me to proceed.

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.

3 participants