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

revert: v4 search autocomplete endpoint #746

Merged
merged 1 commit into from
May 19, 2023

Conversation

amanji
Copy link
Contributor

@amanji amanji commented May 16, 2023

Signed-off-by: Akiff Manji akiff.manji@quartech.com

This PR reverts/removes the v4 search autocomplete endpoint introduced in PR #734

A number of issues were introduced causing confustion in search results and UI artifacts in OrgBook BC

Signed-off-by: Akiff Manji <akiff.manji@quartech.com>
@amanji amanji requested review from esune and WadeBarnes May 16, 2023 18:46
Copy link
Member

@esune esune left a comment

Choose a reason for hiding this comment

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

Looks good to me. @WadeBarnes will wait for your approval to merge and coordinate deploying to test/prod - we will first check-in with affected users to confirm they switched to a different endpoint.

@WadeBarnes
Copy link
Member

@amanji,
Will there be an associated PR for the OrgBook Client? I'm assuming not, as it seems to be using the v3 endpoint.
What affect do the changes to the api/v3 code have on things?

@esune,
Who is tasked with checking in with the affected users? I'll wait for the confirmation before deploying.

@esune
Copy link
Member

esune commented May 17, 2023

@esune, Who is tasked with checking in with the affected users? I'll wait for the confirmation before deploying.

@amanji and/or myself will reach out to them to confirm.

@amanji
Copy link
Contributor Author

amanji commented May 17, 2023

There won't be an associated change to the client. I checked to make sure that we hadn't deployed /v4 autocomplete. /v3 autocomplete will be affected since search suggestions on Topic Name will no longer be effective. Inclusion of that search filter was creating issues with search results. This change should not affect any users of the /v3 or /v4 search topic or credential endpoints.

@WadeBarnes WadeBarnes merged commit 5f2f098 into bcgov:main May 19, 2023
2 checks passed
@WadeBarnes
Copy link
Member

@esune, @amanji, These changes have been deployed to dev and test.

@amanji amanji linked an issue Jun 7, 2023 that may be closed by this pull request
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.

Autocomplete usage producing unusable results
3 participants