search

OHDSI / WebAPI

OHDSI WebAPI contains all OHDSI services that can be called from OHDSI applications
Apache License 2.0
130 stars 169 forks source link

Update PersonService.java #2371

Closed luisleon1894 closed 3 months ago

luisleon1894 commented 6 months ago

Issue: https://github.com/OHDSI/WebAPI/issues/2370

chrisknoll commented 3 months ago

Ok, after discussion on the working group call:

While we acknowledge this isn't part of the CDM spec, the ObservationPeriod class that was implemented as long id so it does make sense that we would read the value from the query via getLong,

So we will accept this PR as-is. Thank you for your contribution!