Skip to content

Fix make_report_error_api usage of Client._project.#3159

Merged
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:fix-error-reporting-project-access
Mar 16, 2017
Merged

Fix make_report_error_api usage of Client._project.#3159
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:fix-error-reporting-project-access

Conversation

@daspecster

Copy link
Copy Markdown
Contributor

Fixes #3158

I'm not sure if Client._project should be a public property in this case but if so I can change it. This PR just correctly access the desired data.

@daspecster daspecster added the api: clouderrorreporting Issues related to the Error Reporting API. label Mar 16, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 16, 2017

@dhermes dhermes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@waprin

waprin commented Mar 16, 2017

Copy link
Copy Markdown
Contributor

LGTM

@DizzeePascall

Copy link
Copy Markdown
Contributor

This is currently blocking my team from using this in production. Could you release an updated version of the pip package?

@lukesneeringer

Copy link
Copy Markdown
Contributor

Thank you @tseaver!

richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
…g-project-access

Fix make_report_error_api usage of Client._project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: clouderrorreporting Issues related to the Error Reporting API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants