Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing documentation for some S3 Object methods #2681

Closed
albertogg opened this issue Mar 29, 2022 · 4 comments · Fixed by #2902
Closed

Missing documentation for some S3 Object methods #2681

albertogg opened this issue Mar 29, 2022 · 4 comments · Fixed by #2902
Labels
documentation This is a problem with documentation.

Comments

@albertogg
Copy link

Describe the issue with documentation
There are some methods that are missing documentation for their optional parameters. I haven't used all methods in the Aws::S3::Object class so I can't talk for all of them, but for example upload_file only reflects 3 optional parameters multipart_threshold, thread_count, and progress_callback but the reality is that it uses multipart upload code and for that reason it inherits all of the options available in that method too. The same thing happens with the copy_to and copy_from methods.

I hope I made the point clear if not I can elaborate further.

Thanks,
Alberto

@albertogg albertogg added the documentation This is a problem with documentation. label Mar 29, 2022
@mullermp
Copy link
Contributor

Thanks for opening an issue. Yes, resource objects (such as Aws::S3::Object) leverage Client API operation methods (such as multipart upload) and have their own documentation. Client API methods are code generated from the API source. I think we can probably add some YARD references to those operations.

@mullermp mullermp added the help wanted We are asking the community to submit a PR to resolve this issue. label Mar 29, 2022
@albertogg
Copy link
Author

albertogg commented Mar 29, 2022

@mullermp if there's an example you can link me to I can try to tackle this. The other thing will be just to add a reference, some basic wording on the YARD documentation something like "This method shares all options from this other method". I think this will be enough.

@mullermp
Copy link
Contributor

Sure, taking upload_file as an example, I think YARD references for @see and using braces {} would be appropriate, perhaps like

Under @param source:

This method takes additional options for {Client#put_object} when file sizes below the multipart threshold. For files larger than the multipart threshold, options for {Client#create_multipart_upload}, {Client#complete_multipart_upload}, and {Client#upload_part} can be provided.

At the very bottom:

@see Client#put_object
@see Client#create_multipart_upload
@see Client#complete_multipart_upload
@see Client#upload_part

To be thorough (if you're up for it), this kind of pattern should be done for all methods in aws-sdk-s3/customizations/.

@mullermp mullermp added no-autoclose and removed help wanted We are asking the community to submit a PR to resolve this issue. labels Mar 27, 2023
@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This is a problem with documentation.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants