Skip to content

[Reporting/Docs] Update Developer API docs#179723

Merged
tsullivan merged 2 commits intoelastic:mainfrom
tsullivan:reporting/update-api-documentation
Apr 1, 2024
Merged

[Reporting/Docs] Update Developer API docs#179723
tsullivan merged 2 commits intoelastic:mainfrom
tsullivan:reporting/update-api-documentation

Conversation

@tsullivan
Copy link
Copy Markdown
Member

@tsullivan tsullivan commented Mar 29, 2024

Summary

Part of documentation updates required for https://github.com/elastic/kibana-team/issues/720

  • Removes outdated info about Angular directives
  • Remove outdated info about the original CSV export type
  • Add explanation of what an Export Type is
  • More detail about the job parameters for each export type
  • Add tips to debug using POST URLs

@tsullivan tsullivan added the release_note:skip Skip the PR/issue when compiling release notes label Mar 29, 2024
@tsullivan tsullivan self-assigned this Mar 29, 2024
@github-actions
Copy link
Copy Markdown
Contributor

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

@tsullivan tsullivan requested a review from a team March 29, 2024 22:00
[float]
=== Generate job URL
To generate a new reporting job, different export types require different `jobParams` that are Rison encoded into a URL
that abide by the following convention: `/api/reporting/generate?jobParams=${rison.encode(jobParams)}`. If you use the
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The stated URL convention isn't completely accurate:

  • There is now an internal version of the path.
  • When generating a report in the Kibana UI, the jobParams are given in the POST payload, not the query string

Copy link
Copy Markdown
Contributor

@sebelga sebelga left a comment

Choose a reason for hiding this comment

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

LGTM 👍

The `export-config` directive will use the browser's current URL and timezone when generating the job
parameters automatically. The `export-config` directive will also grab the height/width of the element
with the `data-shared-items-container` attribute and use this as the dimensions.
<1> The available `layout.id` options for PDF exports are `preserve_layout`, `print`, and `canvas`. These control how dashboard
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we then write the interface like this?

...
layout: {
  id: 'preserve_layout' | 'print' | 'canvas";
  ...

@tsullivan tsullivan enabled auto-merge (squash) April 1, 2024 14:42
@tsullivan tsullivan merged commit d041955 into elastic:main Apr 1, 2024
@kibanamachine kibanamachine added v8.14.0 backport:skip This PR does not require backporting labels Apr 1, 2024
@tsullivan tsullivan deleted the reporting/update-api-documentation branch April 30, 2024 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.14.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants