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

[Canvas] Update Canvas Plugin documentation #65910

Merged
merged 11 commits into from
Jun 29, 2020

Conversation

crob611
Copy link
Contributor

@crob611 crob611 commented May 8, 2020

Updates our README to update to latest Plugin development recommendations.

@crob611 crob611 added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas release_note:skip Skip the PR/issue when compiling release notes labels May 8, 2020
@crob611 crob611 requested a review from a team as a code owner May 8, 2020 19:53
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-canvas (Team:Canvas)

Copy link
Contributor

@clintandrewhall clintandrewhall left a 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.

x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved

> **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.
Copy link
Contributor

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

x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
x-pack/plugins/canvas/README.md Outdated Show resolved Hide resolved
crob611 and others added 9 commits June 29, 2020 09:42
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>
@crob611 crob611 merged commit f47b3e5 into elastic:master Jun 29, 2020
Bamieh pushed a commit to Bamieh/kibana that referenced this pull request Jul 1, 2020
* 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>
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 65910 or prevent reminders by adding the backport:skip label.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 1, 2020
@crob611 crob611 added the backport:skip This commit does not require backporting label Jul 1, 2020
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants