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

Added an endpoint to search diseases from ICD11 #864

Closed
wants to merge 1 commit into from
Closed

Conversation

khavinshankar
Copy link
Member

Added an endpoint (/api/v1/icd) to search for diseases from International Classification of Diseases (ICD), Refer: care_fe#2016

image
image

References:
https://icd.who.int/ct11/icd11_mms/en/release
https://icd.who.int/icdapi

@khavinshankar khavinshankar requested a review from a team as a code owner June 20, 2022 12:21
@vigneshhari
Copy link
Member

I need more context, is the API inaccessible from the front-end, i generally do not prefer using the BE as a proxy if it can be performed directly from the FE.

@khavinshankar
Copy link
Member Author

Yes, we can access this API from front-end but every time we want to access any endpoint, we have to get the bearer token first (authentication is needed). If it is done in the front-end, we have to do the authentication for every client, so thought of going with backend.

@vigneshhari So what are your thoughts Vignesh, should I move this to frontend?

@khavinshankar
Copy link
Member Author

continued in #947

@sainak sainak deleted the icd branch September 6, 2022 11:13
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.

None yet

2 participants