Skip to content

Conversation

@vishaangelova
Copy link
Contributor

@vishaangelova vishaangelova commented Dec 18, 2025

Summary

This PR:

Resolves #3243

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Preview

manage-data/ingest/sample-data.md
manage-data/ingest/upload-data-files.md

@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

Vale Linting Results

Summary: 3 suggestions found

💡 Suggestions (3)
File Line Rule Message
manage-data/ingest/upload-data-files.md 27 Elastic.Wordiness Consider using 'drag' instead of 'drag and drop'.
manage-data/ingest/upload-data-files.md 47 Elastic.Wordiness Consider using 'drag' instead of 'drag and drop'.
manage-data/ingest/upload-data-files.md 61 Elastic.Wordiness Consider using 'drag' instead of 'drag and drop'.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 18, 2025

🔍 Preview links for changed docs

Copy link
Member

@jgowdyelastic jgowdyelastic left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@georgewallace georgewallace left a comment

Choose a reason for hiding this comment

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

Updates look good to me. Thanks for removing the screenshots!

Comment on lines +57 to +69
:::{applies-item} serverless: ga

1. Choose whether to import the data to a new index or to an existing one.
2. Enter a name for the new index or select an existing index from the dropdown.
3. Drag one or more files into the upload area or click **Select or drag and drop a file** to choose files from your computer.

The supported file formats are PDF, TXT, CSV, log files and NDJSON.

After you upload your files, you can inspect the data and make any necessary changes before importing it.

4. Click **Import** to complete the process.

:::
Copy link
Member

Choose a reason for hiding this comment

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

As Stack 9.2 and Serverless are identical, can we combine these into a single tab instead of two? I've seen this pattern elsewhere, for example:

But our cumulative docs guidelines aren't clear about what the recommended pattern is. Do you know? The applies-switch docs also don't cover this.

Copy link
Member

Choose a reason for hiding this comment

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

Ah wait, it is covered here:

You can specify multiple applies_to definitions in a single applies-item using YAML object notation with curly braces {}. This is useful when content applies to multiple deployment types or versions simultaneously.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Internal]: Redesigns to Data Visualizer

5 participants