Skip to content

Conversation

busunkim96
Copy link
Contributor

The Healthcare API has a resource named 'Patient-everything' which is unaccessible because the hyphen makes it an invalid name.

https://healthcare.googleapis.com/$discovery/rest?version=v1

 "Patient-everything": { "response": { "$ref": "HttpBody" }, 

This change replaces the '-' with a '_'.

from googleapiclient import discovery service = discovery.build("healthcare", "v1") service.projects().locations().datasets().fhirStores().fhir().Patient_everything(name='...')

CC @noerog

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 15, 2020
@busunkim96 busunkim96 force-pushed the replace-hyphen-with-underscore branch from 516cf25 to 5b25b32 Compare April 15, 2020 19:52
@noerog
Copy link

noerog commented Apr 17, 2020

Hi @busunkim96, if this could be merged it would allow us to submit some Healthcare API PRs that this is blocking.

@busunkim96 busunkim96 requested a review from plamut April 17, 2020 17:10
@busunkim96 busunkim96 merged commit 8ed729f into googleapis:master Apr 17, 2020
@noerog
Copy link

noerog commented Apr 17, 2020

@busunkim96 After this goes in, do you know when the next client release will happen?

@busunkim96
Copy link
Contributor Author

We have a 'no Friday releases' policy, so I'll cut it on Monday morning. Thank you for your patience!

@noerog
Copy link

noerog commented Apr 17, 2020

Great, thanks.

@mik-laj
Copy link
Contributor

mik-laj commented Apr 17, 2020

@busunkim96 I am very happy that the release will appear soon. I'm waiting. :-D

bmwiedemann added a commit to bmwiedemann/openSUSE that referenced this pull request May 11, 2020
https://build.opensuse.org/request/show/802599 by user mcepl + dimstar_suse - Update to 1.8.2: - Remove apiclient.__version__ gh#googleapis/googleapis#870 - Adding ConnectionError to retry mechanism gh#googleapis/googleapis#558 - replace '-' in method names with '_' gh#googleapis/google-api-python-client#863 - Add missing dependency on oauth2client. - Fix filelist (no CHANGELOG anymore, missing README.md)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

5 participants