-
Notifications
You must be signed in to change notification settings - Fork 22
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
added invoice receipts #12
base: master
Are you sure you want to change the base?
Conversation
🆗 Still missing tests for InvoiceReceipts though right? |
Yes, didn't have time for those. Probably by the end of the week. |
Ok. No problem. Good job. You can merge whenever you want. |
Hi! Any chance of completing this merge anytime soon? |
Any chance of completing this merge anytime soon? |
@donbonifacio would be great to merge this. this is actually missing in the implementation. |
@psousa can you take a look into this conflicts? |
@igorescobar I cloned this repo onto my own account and implemented the invoice receipts part (which is the part I was missing for my project). I did not include any tests. |
@vascosantos cool but @paulosilva86 gave the green light to merge this and somehow It wasn't... |
@igorescobar, I saw that as well, but the merge has been pending for more than 1 year. So I looked at the diffs in the code and copy&pasted on the repository I sent you. |
…. fixed some params on update invoice
…ence_id to sequences. Bumped version.
👓 |
I didn't receive notifications from these comments. So 3 things:
I hope further pull requests will be done in a faster, more orderly way. I'll paste below the changes I've made: Version 0.3.2Added current_invoice_receipt_number and current_invoice_receipt_sequence_id to sequences. Version 0.3.1.1Merged changes from other contributors. Bumped version. Version 0.3.0More big changes on this version:
Version 0.2.5Big changes on this version:
Version 0.2.0Support for 429 RateLimitExceeded code |
No description provided.