Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Update quickstart to include dashboards #6004
Update quickstart to include dashboards #6004
Changes from 19 commits
9da1e70
44a0460
f3a712f
c957ec5
727c3ea
2cfa321
4a14b21
45ca0bd
8843561
daa9d14
cbed625
7a5e749
5e92eec
07d0d9e
b3eb8fc
fbfb6f3
5362726
8d4c032
d4be24f
261fe85
16a6094
7de297f
0f32a7b
38b3d34
e029d92
33ec9c3
749bbc5
4c7a1b2
b108e98
4125309
dc05ebb
19f12b6
ae6ec77
90f36f1
0aa0247
af2a4e2
f373acc
c8f91a3
a8ff097
91d1565
95d643a
f4c63d0
06d3fc9
ddc1d1d
597b11c
5d56f43
0d6e0c2
04b1861
78e2fd5
225bcc7
e920bac
ec73dff
b8fbe95
b799add
50bc9aa
fadfe2b
111e134
15694b1
280e777
5a0f1de
6655687
efe2071
89a9c90
8db94ab
ad3ab64
b063c3c
84641e4
4bdad1e
01f1462
5fac635
2c155f1
4c70673
66df8e5
a2f44bb
26533a2
705980b
9cbe173
cd9dd8d
b33117a
3599f24
f020174
a550004
23a2770
ab83354
9da1940
f0ba91d
102b155
6428084
66827d8
d438f88
503c7d1
1015d8d
11e89d6
884baaa
453b5a8
e9d773f
fc302bc
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
I'd prefer that we keep "guide" for clarity rather than using "quickstart" as a noun. Also, if this is a standalone document, should we use title case here and in the following heading?
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.
Yes to title case
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.
will need to vet this. we don't have any visual regression tests for verifying this. could be accurate now but wrong and we wont know it automatically.
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.
@BSFishy Will you review these lines to make sure the content is accurate? Thanks!
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.
This is accurate. The page extensively uses OUI and properly utilizes the responsive aspects of it, so it should look good on all page sizes. The only disclaimer would be that all within sections are completely owned by plugins, so while the current homepage is responsive, there is not really any guarantee that it is.
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.
Given that the new homepage layout will not be default, but rather will be behind a switch, can we adjust the docs accordingly?
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.
@wbeckler Are you asking that we put in some text around the new Discover toggle (Data Explorer) needing to be activated by the user?
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.
@BSFishy , can you take a look at this and confirm that the homepage plan for 2.12 (keeping current as default and adding a setting to try out the new one) is consistent with these docs changes?
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.
@vagimeli The new homepage has its own separate toggle from the Discover/Data Explorer toggle. I think it defaults to old homepage. I'm also not sure if the PR will make it in for 2.12 as it's still Draft: opensearch-project/OpenSearch-Dashboards#5613
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.
@wbeckler We have specific Discover documentation that I can update when it's ready. this quickstart is intended to cover OpenSearch Dashboards tools generally. Is there any specific information we should remove? Currently, the OpenSearch Playground shows the updates, so I think we need to update the quickstart for relevancy.
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.
@kgcreative Will you review the updates at your availability to make sure the UX topics and screenshots are appropriately captured? Thank you.
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.
How about something like:
"Analyze your data by querying and filtering, viewing results in a table, and diving deeper into documents."
I think this is okay as you have it this is more a suggestion to be clearer. Summarizing and creating don't feel like the right words to describe the action of analyzing. I think the order matters as well - viewing individual documents comes after the table.
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.
When we say visualize data in Discover what are we referring too? The only visualization in Discover is the histogram which isn't new functionality.
Create and editing visualizations still happens in Visualize / VisBuilder.
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.
Revised.
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.
Do we mean "to learn more about using OpenSearch for data ingestion"?
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.
Let's replace this mockups with
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.
Replaced