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

[MD]Address comments from UX sign-off meeting-Datasource list and create page #2625

Merged
merged 3 commits into from
Oct 20, 2022

Conversation

yibow98
Copy link
Contributor

@yibow98 yibow98 commented Oct 19, 2022

Signed-off-by: Yibo Wang yibow@amazon.com

Description

Resolve the meta issue for datasource list page and create page from Multi-datasource UX sign off meeting.

Below are screenshots for UX modification:

  1. Datasource experimental callout wording:

Screen Shot 2022-10-19 at 2 22 44 PM

2. Datasource table column header wording:

Screen Shot 2022-10-19 at 2 27 09 PM

3. Datasource listing page delete button refactor:

Screen Shot 2022-10-19 at 2 27 54 PM

4. Datasource listing page create button in empty state:

Screen Shot 2022-10-19 at 2 28 22 PM

5. Datasource create page spacing between header description and Connection Details:

Screen Shot 2022-10-19 at 2 28 56 PM

6. Datasource create page endpoint field placeholder wording:

Screen Shot 2022-10-19 at 2 29 29 PM

7. Datasource create page toast that fail to save:

Screen Shot 2022-10-19 at 2 30 57 PM

8. Datasource list page fail toast:

Screen Shot 2022-10-19 at 2 32 12 PM

Issues Resolved

#2614
#2615
#2616
#2617
#2618
#2619
#2620
#2626

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
    • yarn test:ftr
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

@yibow98 yibow98 requested a review from a team as a code owner October 19, 2022 21:51
…te page

Signed-off-by: Yibo Wang <yibow@amazon.com>
@yibow98 yibow98 changed the title [MD]Address comments from UX signoff meeting-Datasource list and create page [MD]Address comments from UX sign-off meeting-Datasource list and create page Oct 19, 2022
@yibow98 yibow98 self-assigned this Oct 19, 2022
@yibow98 yibow98 added multiple datasource multiple datasource project v2.4.0 'Issues and PRs related to version v2.4.0' ux / ui Improvements or additions to user experience, flows, components, UI elements backport 2.x labels Oct 19, 2022
@KrooshalUX
Copy link

KrooshalUX commented Oct 20, 2022

1-7 LGTM

In regards to 8 - If a user has not created any data source connections yet, it is incorrect to present an error toast. When working through these use cases with @kristenTian, I believe it was agreed that the danger toast would only be used when there was a server error preventing data source connections to load. Given the timeline, I can mark this for enhancement in 2.5, unless its something you think is a reasonable fix for 2.4.

@yibow98
Copy link
Contributor Author

yibow98 commented Oct 20, 2022

1-7 LGTM

#8 - If a user has not created any data source connections yet, it is incorrect to present an error toast. When working through these use cases with @kristenTian, I believe it was agreed that the danger toast would only be used when there was a server error preventing data source connections to load. Given the timeline, I can mark this for enhancement in 2.5, unless its something you think is a reasonable fix for 2.4.

@KrooshalUX Thanks for your comment! The toast will not show up when a user has not created any data source connections. In the screenshot, I just want to show the toast has been changed to danger state. Thus, I first shut down the server and tried to create a datasource, then it showed an error toast. So currently design matches your statement "The danger toast would only be used when there was a server error preventing data source connections to load."

Copy link

@KrooshalUX KrooshalUX left a comment

Choose a reason for hiding this comment

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

Upon clarification from @yibow98 , all items 1-8 look good and have UX approval.
Thanks team for your continued efforts!

@yibow98
Copy link
Contributor Author

yibow98 commented Oct 20, 2022

Upon clarification from @yibow98 , all items 1-8 look good and have UX approval. Thanks team for your continued efforts!

Thanks @KrooshalUX for your review and all your efforts!

@zhongnansu zhongnansu merged commit fb4bdfa into opensearch-project:main Oct 20, 2022
@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.x 2.x
# Navigate to the new working tree
cd .worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-2625-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 fb4bdfa7bfd10878481814862fc794debeac272e
# Push it to GitHub
git push --set-upstream origin backport/backport-2625-to-2.x
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-2625-to-2.x.

@opensearch-trigger-bot
Copy link
Contributor

The backport to 2.x failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.x 2.x
# Navigate to the new working tree
cd .worktrees/backport-2.x
# Create a new branch
git switch --create backport/backport-2625-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 fb4bdfa7bfd10878481814862fc794debeac272e
# Push it to GitHub
git push --set-upstream origin backport/backport-2625-to-2.x
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.x

Then, create a pull request where the base branch is 2.x and the compare/head branch is backport/backport-2625-to-2.x.

zhongnansu pushed a commit to zhongnansu/OpenSearch-Dashboards that referenced this pull request Oct 21, 2022
…te page (opensearch-project#2625)

Signed-off-by: Yibo Wang <yibow@amazon.com>
(cherry picked from commit fb4bdfa)
zhongnansu pushed a commit to zhongnansu/OpenSearch-Dashboards that referenced this pull request Oct 21, 2022
…te page (opensearch-project#2625)

Signed-off-by: Yibo Wang <yibow@amazon.com>
(cherry picked from commit fb4bdfa)
Signed-off-by: Su <szhongna@amazon.com>
zhongnansu added a commit that referenced this pull request Oct 24, 2022
…te page (#2625) (#2642)

Signed-off-by: Yibo Wang <yibow@amazon.com>
(cherry picked from commit fb4bdfa)
Signed-off-by: Su <szhongna@amazon.com>

Signed-off-by: Yibo Wang <yibow@amazon.com>
Signed-off-by: Su <szhongna@amazon.com>
Co-authored-by: Yibo Wang <109543558+yibow98@users.noreply.github.com>
@AMoo-Miki AMoo-Miki added the enhancement New feature or request label Nov 5, 2022
sipopo pushed a commit to sipopo/OpenSearch-Dashboards that referenced this pull request Dec 16, 2022
…te page (opensearch-project#2625)

Signed-off-by: Yibo Wang <yibow@amazon.com>
Signed-off-by: Sergey V. Osipov <sipopo@yandex.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x enhancement New feature or request multiple datasource multiple datasource project ux / ui Improvements or additions to user experience, flows, components, UI elements v2.4.0 'Issues and PRs related to version v2.4.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants