redguava / cliniko-api

The API for Cliniko
74 stars 39 forks source link

Adding staging internal admin product integration #281

Closed lionsdwarf closed 4 years ago

lionsdwarf commented 4 years ago

Hello,

We would like to white-list one more integration URL for our staging environment. The URL we want to support is https://staging.findoctave.com/cliniko

Please let me know how best to proceed.

Thanks!

hagen commented 4 years ago

Hi there - I think instead of adding your staging environments to our production environment (which is what we'd be doing here), you could just test that your staging env works with tests for https://staging.findoctave.com/cliniko?patient_id={patient_id}?

i.e. that the request comes from Cliniko itself shouldn't be important; rather that your staging environment correctly responds to requests like that above is the thing you want to check, right?

hagen commented 4 years ago

Closing - no reply. Assume all's good 🤙 Re-open if required.