Skip to content

Conversation

@niccokunzmann
Copy link

Hi,

I am about to use your lib and I would like to improve the documentation of it.
This is a little taster to get to know each other. I do not want to invest a lot of time if you do not like the changes.
I would open an issue with changes I would like to make if you accept changes.

What do you think?
Greetings,
Nicco

@jayaddison
Copy link
Owner

Thanks @niccokunzmann - I'll read through this and your feature pull request within the next few days or so. As you've discovered, I tend to turn GitHub Actions off for many of my repositories. I'll run tests/linting locally during code review and may also turn GitHub Actions on again soon.

@niccokunzmann
Copy link
Author

Thanks!

In the future, I also want to create transfers.
So, there are more features to come.

Comment on lines +13 to +17
After installation, you should be able to import the package.

```python
import pywisetransfer
```
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's omit this; the package is a simple one, and provided that the installation process succeeds, this should too.

We should consider including it in complete code examples, however also avoid too much repetition.

Comment on lines +45 to +53
```python
client = pywisetransfer.Client(api_key="your-api-key-here", environment="live")
```

- Create a `Client` object with your API key for the `sandbox` environment:

```python
client = pywisetransfer.Client(api_key="your-api-key-here")
```
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally I think we should make this readme friendly to newcomers, but I also want to balance that against how safe and mature the code is (I'll be more comfortable if we can reduce the number of external dependencies, for example).

Even when we're very confident about the safety and quality of the code, we should still write the documentation in a way that requires people to be very careful to enable production usage. So: at the very least we should present the sandbox environment first -- and perhaps we should remove the live example entirely.

name: Law and Daughters 6423
```

### Receive Money
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Receive Money
### Account Details

@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