Skip to content

Mention that cors property has a setter. - #2623

Merged
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:update-storage-cors-docstring
Oct 27, 2016
Merged

daspecster merged 1 commit into
googleapis:masterfrom
daspecster:update-storage-cors-docstring

Conversation

@daspecster

Copy link
Copy Markdown
Contributor

Bucket CORS property docs only say retrieve but it actually has a setter as well.

See #1838.

@daspecster daspecster added docs api: storage Issues related to the Cloud Storage API. labels Oct 26, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 26, 2016
@mlissner

Copy link
Copy Markdown
Contributor

Do you want to add docs for the lifecycle_rules setter too, as in #2610?

@daspecster

Copy link
Copy Markdown
Contributor Author

@mlissner, I think that will be a separate PR actually.

https://cloud.google.com/storage/docs/json_api/v1/buckets

:setter: Set CORS policies for this bucket.
:getter: Gets the CORS policies for this bucket.

This comment was marked as spam.

@tseaver

tseaver commented Oct 26, 2016

Copy link
Copy Markdown
Contributor

@dhermes There is a SO answer which shows :getter: and :setter: being rendered.

@daspecster

Copy link
Copy Markdown
Contributor Author

I'm generating the docs real quick and I'll screenshot it.

@daspecster

Copy link
Copy Markdown
Contributor Author

screen shot 2016-10-26 at 4 12 17 pm

@dhermes

dhermes commented Oct 26, 2016

Copy link
Copy Markdown
Contributor

Got it, so they aren't official directives but they are "generally supported" syntax

@daspecster

Copy link
Copy Markdown
Contributor Author

@dhermes @tseaver is this good to merge?

@dhermes

dhermes commented Oct 27, 2016

Copy link
Copy Markdown
Contributor

👍

@daspecster
daspecster merged commit f723397 into googleapis:master Oct 27, 2016
@daspecster
daspecster deleted the update-storage-cors-docstring branch October 27, 2016 16:03
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
…s-docstring

Mention that cors property has a setter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants