-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Canvas] Update Canvas Plugin documentation #65910
Conversation
Pinging @elastic/kibana-canvas (Team:Canvas) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good. I'd like to make a suggestion, but it's a bit of a pain in the neck: can we link to some of these TS types in the code examples? I think it would make it a lot easier to grep. Might be a good automation, perhaps. Just a suggestion.
Great work.
|
||
> **FATAL** Error: Unmet requirement "interpreter" for plugin "your_plugin_name" | ||
|
||
then your plugin was likely created to work on a previous version of Kibana. Starting with version 7.8, the plugin system was redesigned and caused breaking changes to these earlier plugins. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: add link to the release notes on breaking changes
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
* Update Canvas Plugin documentation * Small fix * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> * Update x-pack/plugins/canvas/README.md Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com> Co-authored-by: Clint Andrew Hall <clint@clintandrewhall.com>
Friendly reminder: Looks like this PR hasn’t been backported yet. |
Updates our README to update to latest Plugin development recommendations.