Skip to content

Conversation

@niccokunzmann
Copy link

@niccokunzmann niccokunzmann commented Dec 26, 2024

Hi, thanks for this module! I am looking forward to using it. However, I would like to add a bit more to it before.
The earlier the features are merged, the earlier I can add on top of them :)

Changes:

  • add client.transfers.list
  • add client.transfers.get
  • ignore .venv (VSCode)
  • create tests
  • use the new recorder feature of responses to construct an API and keep the tests cases small and simple

See #32.

@niccokunzmann
Copy link
Author

Hm. I do not see the tests running ...

@jayaddison
Copy link
Owner

use the new recorder feature of responses to construct an API and keep the tests cases small and simple

Hi @niccokunzmann - I'd prefer using minimal, non-recorded snippets that preferably don't include any non-essential fields and information; for example, using sample/made-up values.

@niccokunzmann niccokunzmann mentioned this pull request Jan 6, 2025
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.

2 participants