Skip to content

Panel: removes deprecated componets #503

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Panel: removes deprecated componets #503

wants to merge 2 commits into from

Conversation

hugohaggmark
Copy link
Contributor

This PR will remove TimeSeries, TooltipPlugin and ZoomPlugin etc from @grafana/ui. This PR replaces them with an Table example until there is a valid migration path.

Fixes: #502

@hugohaggmark hugohaggmark requested review from a team as code owners April 7, 2025 09:50
@hugohaggmark hugohaggmark requested review from briangann and removed request for a team April 7, 2025 09:50
@hugohaggmark hugohaggmark requested review from wbrowne and andresmgot and removed request for a team April 7, 2025 09:50
andresmgot
andresmgot reviewed Apr 7, 2025
View reviewed changes
Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

so if I read the thread correctly, the idea is to keep TimeSeries and the tooltip component?

In any case, the README of this example would need to be updated too.

@hugohaggmark
Copy link
Contributor Author

so if I read the thread correctly, the idea is to keep TimeSeries and the tooltip component?

In any case, the README of this example would need to be updated too.

Awesome catch, I missed that 👍

@hugohaggmark hugohaggmark requested a review from andresmgot April 7, 2025 11:15
andresmgot
andresmgot reviewed Apr 8, 2025
View reviewed changes
Copy link
Contributor

@andresmgot andresmgot left a comment

Choose a reason for hiding this comment

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

Should we wait until we know if we can keep using TimeSeries?

@hugohaggmark hugohaggmark moved this from 📬 Triage to 🔬 In review in Plugins Platform / Grafana Community Apr 8, 2025
@hugohaggmark hugohaggmark self-assigned this Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andresmgot andresmgot andresmgot left review comments

@briangann briangann Awaiting requested review from briangann briangann is a code owner automatically assigned from grafana/plugins-platform-frontend

@wbrowne wbrowne Awaiting requested review from wbrowne wbrowne is a code owner automatically assigned from grafana/plugins-platform-backend

At least 1 approving review is required to merge this pull request.

Assignees

@hugohaggmark hugohaggmark

Labels
chore
Projects
Plugins Platform / Grafana Community
Status: 🔬 In review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Replace TimeSeries, TooltipPlugin and ZoomPlugin
2 participants
@hugohaggmark @andresmgot