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

Fixed map function call that causes problems in Grafana 10 (bump to 1.1.1) #19

Merged
merged 1 commit into from
Jul 21, 2023

Conversation

Sawy7
Copy link
Contributor

@Sawy7 Sawy7 commented Jul 4, 2023

I changed one line in dataParser.ts so that this panel plugin now works in the latest version. I verified on my local instance that it didn't work before and now works just fine. Also bumped the minor version to 1.1.1. Don't know if that is desirable.

This fixes issue #18

Thanks for your work on this amazing plugin!

@aned
Copy link

aned commented Jul 20, 2023

@KatrinaTurner can this be reviewed please?

@KatrinaTurner
Copy link
Collaborator

Thank you for the PR! Taking a look!

@KatrinaTurner
Copy link
Collaborator

KatrinaTurner commented Jul 21, 2023

Hi @Sawy7 ! Can you actually change this to request it is merged into netsage-project/1.1.1 branch instead of master please? I just created it for you :) After that I can approve, merge and re-release

@Sawy7 Sawy7 changed the base branch from master to 1.1.1 July 21, 2023 18:20
@Sawy7
Copy link
Contributor Author

Sawy7 commented Jul 21, 2023

Hey @KatrinaTurner, I just changed the destination branch to 1.1.1. Thanks for your time!

@KatrinaTurner KatrinaTurner merged commit 7b21d81 into netsage-project:1.1.1 Jul 21, 2023
@AbrahamPrez25
Copy link

Thank you both for this fix. When will it be available in the master branch? @KatrinaTurner

@KatrinaTurner
Copy link
Collaborator

I usually wait until Grafana accepts the update, but since it's so simple I can merge now :)

@AbrahamPrez25
Copy link

I saw you already did! Thank you! I asked because I thought that it would automatically show up in Grafana this new version. But I guess it won't happen until they accept the update. How long does this usually take? The only way to check it is to look at the site https://grafana.com/grafana/plugins/netsage-sankey-panel/ ? Thanks in advance, I am a bit new to this "plugin development" face of Grafana, your work helps a lot, Katrina ❤️

@KatrinaTurner
Copy link
Collaborator

KatrinaTurner commented Jul 26, 2023 via email

@KatrinaTurner
Copy link
Collaborator

KatrinaTurner commented Jul 26, 2023 via email

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.

4 participants