An optional 'id' argument for Manager.save method.

[vadim-pavlov]

The new argument should enable Xero API methods utilizing Resource ID URI parameter in POST requests. Such as deletion of payments and bank transactions:
https://developer.xero.com/documentation/api/payments#POST
https://developer.xero.com/documentation/api/banktransactions#POST

[freakboy3742]

Thanks for the suggestion; but I'm not convinced this is the right approach to this API. If the use case is deletion, then we should add a delete entry point on objects (or, at least, on objects that support it)

[vadim-pavlov]

I've changed the solution based on your feedback. I added a PaymentManager with a custom delete method.

[vadim-pavlov]

I guess a better alternative could be to pass a delete_strategy callable to the BaseManager constructor, in this case there will be no need to define a manager class for every entity that has non-standard API.

[freakboy3742]

Apologies for the delay in reviewing this; the PR appears to have drifted out of sync with main. Could I ask you to merge it with main?

[vadim-pavlov]

Not a problem @freakboy3742. I've merged it with the latest main.

[freakboy3742]

Looks like I have some additional housekeeping to do (dropping Python 3.6 support); I'll add that to my todo list, then review this for you.

[freakboy3742]

Looks good; apologies again for the late review, and thanks for the contribution!