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 Test in README.rst to reflect API changes #119

Closed
wants to merge 1 commit into from

Conversation

naidoo88
Copy link

@naidoo88 naidoo88 commented Sep 4, 2024

Including the target as the third argument in the cds.retrieve() method does not work with 0.7.2, returning the error:

AttributeError: 'str' object has no attribute 'download'

The included changes fix this issue, pointing the user to the appropriate syntax.

Including the target as the third argument in the cds.retrieve() method does not work with 0.7.2, returning the error:

AttributeError: 'str' object has no attribute 'download'

The included changes fix this issue, pointing the user to the appropriate syntax.
@FussyDuck
Copy link

FussyDuck commented Sep 4, 2024

CLA assistant check
All committers have signed the CLA.

@naidoo88
Copy link
Author

naidoo88 commented Sep 4, 2024

Realising the error was in the inclusion of .download(), inferred from inspecting the API request generated on the web order portal. Removing this fixes the aforementioned issue, and as such the current documentation is correct!

@naidoo88 naidoo88 closed this Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants