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

Update base.py #454

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update base.py #454

wants to merge 1 commit into from

Conversation

alistvt
Copy link

@alistvt alistvt commented Nov 30, 2022

adding param explanations for files_upload

Checklist

General Contributing

  • Have you read the Code of Conduct and signed the CLA?

Is This a Code Change?

  • Non-code related change (markdown/git settings etc)
  • SDK Code Change
  • Example/Test Code Change

Validation

  • Does tox pass?
  • Do the tests pass?

adding param explanations for `files_upload`
@CLAassistant
Copy link

CLAassistant commented Nov 30, 2022

CLA assistant check
All committers have signed the CLA.

@greg-db
Copy link
Contributor

greg-db commented Nov 30, 2022

Thanks! That file is actually autogenerated though, so that change wouldn't be persisted across releases. This would need to be addressed in the generator.

I'll ask the team to investigate why these parameters aren't always being automatically included in the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants