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

Refactor medic code querying the freetext views to hit the API/Nouveau index instead #9692

Open
m5r opened this issue Dec 4, 2024 · 0 comments
Labels
Type: Improvement Make something better

Comments

@m5r
Copy link
Member

m5r commented Dec 4, 2024

What feature do you want to improve?
Code querying the freetext views will fail once we ship Nouveau (#9542) since we're deleting those views and replacing them with Nouveau indexes.

Describe the improvement you'd like
The easy fix is to replace the URL to those views by the URL to the replacing indexes and change the query to satisfy Nouveau's query format. The impacted repos might want to support both querying the old views and the new indexes while deployments upgrade. This can be done by checking for the existence of the views or by looking up the CHT instance's version.

The ideal fix is for those impacted repos to query the CHT using cht-datasource and/or relying only on APIs exposed by the CHT instead.

Additional context
This should include every repo outside of cht-core that needs updating.
https://github.com/search?q=org%3Amedic+freetext+%28language%3AJavaScript+OR+language%3ATypeScript%29++NOT+repo%3Amedic%2Fcht-core+NOT+repo%3Amedic%2Fcht-accelerator+NOT+repo%3Amedic%2Fcht-docs+NOT+repo%3Amedic%2Fchromium-crosswalk&type=code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Improvement Make something better
Projects
Status: Todo
Status: Todo
Development

No branches or pull requests

1 participant