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][Docs] Fixes Canvas doc headers #40921

Merged
merged 1 commit into from
Jul 11, 2019

Conversation

cqliu1
Copy link
Contributor

@cqliu1 cqliu1 commented Jul 11, 2019

Summary

Follow up to #38300.
[skip-ci]

@tsullivan brought to my attention that there were a bunch of errors from the Canvas docs when trying to build the Kibana docs. This fixes those errors.

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

- [ ] This was checked for cross-browser compatibility, including a check against IE11
- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support
- [ ] Documentation was added for features that require explanation or tutorials
- [ ] Unit or functional tests were updated or added to match the most common scenarios
- [ ] This was checked for keyboard-only and screenreader accessibility

For maintainers

@cqliu1 cqliu1 added impact:critical This issue should be addressed immediately due to a critical level of impact on the product. loe:small Small Level of Effort release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas Team:Docs labels Jul 11, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-canvas

Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

LGTM
I helped with the regex to add all the newlines and [floats]

@cqliu1 cqliu1 force-pushed the docs/fix-canvas-docs branch from ac97a08 to 0e3a77c Compare July 11, 2019 22:36
@cqliu1 cqliu1 merged commit 5103e5b into elastic:master Jul 11, 2019
@cqliu1 cqliu1 deleted the docs/fix-canvas-docs branch July 11, 2019 22:36
cqliu1 added a commit to cqliu1/kibana that referenced this pull request Jul 11, 2019
cqliu1 added a commit to cqliu1/kibana that referenced this pull request Jul 11, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

cqliu1 added a commit that referenced this pull request Jul 12, 2019
* [Canvas][Docs] Merge Canvas function references (#38300)

* Synced up docs with function defs

* Updated docs with changes from PR #37305

* Merged canvas function ref into a single doc

* Fixed arg order

* Fixed typo

* Added alphabet links

* Added alphabet headers

* Removed B header

* Added missing args from

* Added edits from PR #37614

* Updated containerStyle

* Removed metafields. ESSQL doesn't support retrieving metafields

* Edits to function copy

* More edits

* More edits

* Final round of edits

* Fixed i18n errors

* Addressing feedback

* Fixed jest test

* Fixed missing import

* Fixed i18n error

* Restored metaFields arg in esdocs

* Extracted i18n string constants

* Fixed i18n errors

* Updated translation files

* Fixes Canvas doc headers (#40921)
cqliu1 added a commit that referenced this pull request Jul 12, 2019
* [Canvas][Docs] Merge Canvas function references (#38300)

    * Synced up docs with function defs

    * Updated docs with changes from PR #37305

    * Merged canvas function ref into a single doc

    * Fixed arg order

    * Fixed typo

    * Added alphabet links

    * Added alphabet headers

    * Removed B header

    * Added missing args from

    * Added edits from PR #37614

    * Updated containerStyle

    * Removed metafields. ESSQL doesn't support retrieving metafields

    * Edits to function copy

    * More edits

    * More edits

    * Final round of edits

    * Fixed i18n errors

    * Addressing feedback

    * Fixed jest test

    * Fixed missing import

    * Fixed i18n error

    * Restored metaFields arg in esdocs

    * Extracted i18n string constants

    * Fixed i18n errors

    * Updated translation files

* Fixes Canvas doc headers (#40921)

* Fixed imports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact:critical This issue should be addressed immediately due to a critical level of impact on the product. loe:small Small Level of Effort release_note:skip Skip the PR/issue when compiling release notes Team:Docs Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v7.3.0 v7.4.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants