-
Notifications
You must be signed in to change notification settings - Fork 30
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
[Docs] Add Ingest tools overview and Upload data files #526
[Docs] Add Ingest tools overview and Upload data files #526
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🦘
Nice work! I checked all of the links and they work swimmingly.
For your questions:
Since we decided that a table would work best for the overview, I'm not sure how to incorporate https://www.elastic.co/guide/en/cloud/current/ec-cloud-ingest-data.html on this page. I don't think all of its content fits here based on the new IA. Maybe it could go in the reference section? I'm open to any ideas!!
That "Adding data to Elasticsearch" page is now extinct, but I think it would be helpful to say something in the first paragraph of your tools page, something like, "Refer to our [[Ingest]] overview for some guidelines to help you select the optimal tool for your use case."
The Upload data files page was originally nested under the Ingest tools overview, but I think it's better suited as a sibling in the overall Manage data/Ingest section. What do you think?
I like the new location!
One suggestion: The order of the tools isn't really obvious to me. I'm not sure if they should be alphabetical or just left as is, but if indeed the order is sort of random, perhaps bump "Integrations" further up in the list, since those are really the preferred mechanism for ingesting data. Just a thought.
Thanks for reviewing and answering my questions, @kilfoyle! I've incorporated your suggestions. The order of the tools is indeed random, so I can definitely move things around if they make sense in a different order. Let me know 🙂 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🦕
Looks good! Thanks for moving "integrations" to the top.
Closes: #327 and #444
Previews: Ingest tools overview and Upload data files
Summary
Questions