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

[Backport 2.x] Adding error message in Search Comparison Tool #305

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Oct 4, 2023

Backport 6fb31c8 from #267.

* adding error message in search comparison tool

Signed-off-by: Sean Li <lnse@amazon.com>

* updating test snapshots

Signed-off-by: Sean Li <lnse@amazon.com>

* adding tests for query error

Signed-off-by: Sean Li <lnse@amazon.com>

* updating tests

Signed-off-by: Sean Li <lnse@amazon.com>

* refactoring QueryError, adding error message to result panel

Signed-off-by: Sean Li <lnse@amazon.com>

* moving helper text to helpText prop

Signed-off-by: Sean Li <lnse@amazon.com>

* adding tests and addressing comments

Signed-off-by: Sean Li <lnse@amazon.com>

* adding more tests, fixing state management, addressing comments

Signed-off-by: Sean Li <lnse@amazon.com>

* reverting changes to validateQuery, addressing comments

Signed-off-by: Sean Li <lnse@amazon.com>

---------

Signed-off-by: Sean Li <lnse@amazon.com>
(cherry picked from commit 6fb31c8)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@sejli sejli merged commit 5291c80 into 2.x Oct 4, 2023
2 checks passed
@github-actions github-actions bot deleted the backport-267-to-2.x branch October 4, 2023 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant