Review SmartAPI registration update and merged PR and then redeploy ICEES+ API instances

ExposuresProvider / icees-api

MIT License

2 stars 8 forks source link

Review SmartAPI registration update and merged PR and then redeploy ICEES+ API instances #253

Closed karafecho closed 1 year ago

karafecho commented 1 year ago

Max and I updated a bunch of SmartAPI registrations over the holiday break, including this one: https://github.com/ExposuresProvider/icees-api/pull/251#pullrequestreview-1225041490. This issue is to request that you review the change and, if no additional changes are required, redeploy the ICEES+ API instances.

hyi commented 1 year ago

@karafecho I think this ticket can be closed now that the redeployed ICEES+ API instances have been verified.

karafecho commented 1 year ago

Agreed. I didn't test the production instances of the ICEES+ APIS, but I did test the four development instances (asthma, pcd, dili, covid) by creating a cohort and running a feature association functionality. All looks fine, so closing ticket.