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

Update link to plugins docs in scene-app.md #339

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

josmperez
Copy link

No description provided.

Copy link
Member

@dprokop dprokop left a comment

Choose a reason for hiding this comment

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

I don't think the proposed link is valid in this context. It drops a person right into the plugin creation guide, without explaining what the Grafana plugin even is.

@josmperez
Copy link
Author

josmperez commented Sep 14, 2023

I don't think the proposed link is valid in this context.

To be clear, it's not a proposed link. It updates the old link to the correct new URL. I'm all for adding context to the doc, but today I just want to get the address correct.

I'm heads down on the plugin site, but if it would be helpful I can take a look at this doc next week and offer suggestions.

@dprokop
Copy link
Member

dprokop commented Sep 18, 2023

but today I just want to get the address correct

But this address ain't correct. I mean - it's fine, but it's throwing people into deep water, straight into plugin creation, without letting them know what plugins are. While the old one is basically Grafana plugin developer’s guide, which is an intro to what plugins are and how they can help.

@dprokop
Copy link
Member

dprokop commented Sep 18, 2023

I'm heads down on the plugin site, but if it would be helpful I can take a look at this doc next week and offer suggestions.

Yes, definitely, please!

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.

None yet

2 participants