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

Updates file upload formats in Kibana guide #195880

Merged

Conversation

kosabogi
Copy link
Contributor

@kosabogi kosabogi commented Oct 11, 2024

Overview

This PR updates the Upload a file section in the Add data page to reflect the changes to the supported file formats in the Data Visualizer.

Related issue:

https://github.com/elastic/search-docs-team/issues/189#issuecomment-2399079270

Preview

@kosabogi kosabogi added Team:Docs release_note:skip Skip the PR/issue when compiling release notes v9.0.0 v8.16.0 backport:version Backport to applied version labels labels Oct 11, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

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

Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

Thx! One nit and one question: do we need to update the screenshot too?

Comment on lines 83 to 84
- CSV

Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: don't need spaces in between and should use * instead of - for consistency :)

Copy link
Contributor Author

@kosabogi kosabogi Oct 11, 2024

Choose a reason for hiding this comment

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

Thanks for the review! I fixed the formatting issue in a new commit.
The UI hasn't changed so far, so I think we can't update the screenshot here:

Screenshot 2024-10-11 at 15 04 26

@leemthompo
Copy link
Contributor

leemthompo commented Oct 11, 2024

Screenshot 2024-10-11 at 15 21 02

I just pulled this from a serverless deployment where the changes are already live (it's probably live on 8.16.0 SNAPSHOT and definitely main too). That screenshot should do the trick :)

@leemthompo
Copy link
Contributor

@kosabogi I see you updated in the other PR, the screenshots should be identical so you can replicate here :)

Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

Approving, pending screenshot update! Thanks :)

@kosabogi kosabogi merged commit 94ee8d5 into elastic:main Oct 14, 2024
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11324101019

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 14, 2024
### Overview

This PR updates the **Upload a file** section in the **Add data** page
to reflect the changes to the supported file formats in the Data
Visualizer.

### Related issue:

elastic/search-docs-team#189 (comment)

### Preview
(cherry picked from commit 94ee8d5)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Oct 14, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Updates file upload formats in Kibana guide
(#195880)](#195880)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"kosabogi","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-10-14T08:48:36Z","message":"Updates
file upload formats in Kibana guide (#195880)\n\n###
Overview\r\n\r\nThis PR updates the **Upload a file** section in the
**Add data** page\r\nto reflect the changes to the supported file
formats in the Data\r\nVisualizer.\r\n\r\n### Related issue:
\r\n\r\n\r\nhttps://github.com/elastic/search-docs-team/issues/189#issuecomment-2399079270\r\n\r\n###
Preview","sha":"94ee8d598fe9fdf388a49e48580c2db58e85df2b","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Docs","release_note:skip","v9.0.0","v8.16.0","backport:version"],"title":"Updates
file upload formats in Kibana
guide","number":195880,"url":"https://github.com/elastic/kibana/pull/195880","mergeCommit":{"message":"Updates
file upload formats in Kibana guide (#195880)\n\n###
Overview\r\n\r\nThis PR updates the **Upload a file** section in the
**Add data** page\r\nto reflect the changes to the supported file
formats in the Data\r\nVisualizer.\r\n\r\n### Related issue:
\r\n\r\n\r\nhttps://github.com/elastic/search-docs-team/issues/189#issuecomment-2399079270\r\n\r\n###
Preview","sha":"94ee8d598fe9fdf388a49e48580c2db58e85df2b"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/195880","number":195880,"mergeCommit":{"message":"Updates
file upload formats in Kibana guide (#195880)\n\n###
Overview\r\n\r\nThis PR updates the **Upload a file** section in the
**Add data** page\r\nto reflect the changes to the supported file
formats in the Data\r\nVisualizer.\r\n\r\n### Related issue:
\r\n\r\n\r\nhttps://github.com/elastic/search-docs-team/issues/189#issuecomment-2399079270\r\n\r\n###
Preview","sha":"94ee8d598fe9fdf388a49e48580c2db58e85df2b"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: kosabogi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Docs v8.16.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants