Skip to content
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.

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

fix: support grafana 8.3.x #141

Merged
merged 1 commit into from
Feb 24, 2022

Conversation

geryepes
Copy link

Upstream changes related to this PR
grafana/grafana@7319efe

@geryepes geryepes mentioned this pull request Jan 27, 2022
@geryepes
Copy link
Author

Resolves #139

@yggdrasil-tynor
Copy link

@geryepes For me it looks good. However it would be better for a regular contributor to make the review, as i haven't worked on grafana plugins.

Nice work by the way :)

@vicioussn
Copy link

Anyone? Please merge it.

@Gauravshah
Copy link
Member

Gauravshah commented Feb 4, 2022 via email

@yggdrasil-tynor
Copy link

@Gauravshah would you be happy to review it? :-)

@ryanpxyz
Copy link

Hello,

... a quick turnaround on this would be very welcomed!

Thanks,

Phil.

Copy link
Member

@Gauravshah Gauravshah left a comment

Choose a reason for hiding this comment

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

Thank you for putting in PR and also linking the upstream grafana change

dist/datasource.js Outdated Show resolved Hide resolved
@ryanpxyz
Copy link

Good morning,

Is there a status update? Are there plans to release a 0.0.6?

@geryepes geryepes force-pushed the fix_support_grafana_8.3.x branch from 14b64a6 to ef6ba7b Compare February 16, 2022 17:32
@geryepes geryepes requested a review from Gauravshah February 16, 2022 17:37
Copy link
Member

@Gauravshah Gauravshah left a comment

Choose a reason for hiding this comment

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

@ShilpaSivanesan can we merge this ?

@ShilpaSivanesan
Copy link
Contributor

@ShilpaSivanesan can we merge this ?

Sure, will validate and merge

@yggdrasil-tynor
Copy link

@Gauravshah / @ShilpaSivanesan sorry to bump you guys. But can this please be merged, so we can use this great plugin with grafana 8.4.0 👍

@ryanpxyz
Copy link

Good morning,

Is there a status update? Are there plans to release a 0.0.6?

@ShilpaSivanesan
Copy link
Contributor

Thanks for your contribution @geryepes . Really appreciate it.

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.

6 participants