ExposuresProvider / icees-api

MIT License
2 stars 8 forks source link

remove knowledge graph related endpoints #255

Closed hyi closed 1 year ago

hyi commented 1 year ago

This PR removes knowledge graph related endpoints and cleanup code from ICEES+ API code base now that we have a separate ICEES KG service.

hyi commented 1 year ago

@karafecho Updated ICEES+ API services with knowledge graph related endpoints removed can be tested in all development instances. If everything looks good, I'll update production instances. Just let me know whether the development instances look good once you have a chance to test them out.

karafecho commented 1 year ago

Looks fine to me, too. Note that I not only verified that the KG endpoints (and only the KG endpoints) had been successfully removed, but I tested the four development instances of the ICEES+ API (asthma, pcd, dili, covid) by creating a cohort and running a feature association functionality, and all looks fine, so this PR can be merged. Thanks, Hong!