Skip to content

Conversation

@varunajmera0
Copy link
Contributor

Here is my implementation:-
I updated the invoice file methods like -

  1. issue
  2. delete
  3. send/ resend notification ( - payment link)
  4. edit
  5. cancel ( - payment link)
    You can use these methods on payment link.

A new Settlement API is done.

… added in invoice file and new api i.e. settlement api
…ssue, edit method added in invoice file and new api i.e. settlement api
@varunajmera0 varunajmera0 changed the title Implementation - Settlement API, issue, cancel, send/ resend notification methods Implementation - Settlement API, issue, cancel, send/ resend notification, delete, edit methods Feb 5, 2019
@amitmahbubani amitmahbubani self-assigned this Feb 24, 2019
@captn3m0
Copy link
Contributor

@varunajmera0 Thanks for the Pull Request! We're getting this reviewed internally. 🔜

Copy link
Contributor

@jitendra-1217 jitendra-1217 left a comment

Choose a reason for hiding this comment

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

A few minor things need addressing, please check below.

Request to please add tests for new endpoints, for invoice you can just add new test cases in TestClientInvoice.py. For settlements request to do it same way by adding new test class.

…point and test file is created for settlement and test function is done in test invoice file
…point and test file is created for settlement and test function is done in test invoice file
…ncel, delete and issue endpoint and test file is created for settlement and test function is done in test invoice file
- Create payment link

```py
DATA = {"customer": { "name": "Varun Test",
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to fix formatting and sample data in all these snippets.

invoice_id : Id for delete the invoice
data : Dictionary having keys using which invoice have to be updated
Returns:
{
Copy link
Contributor

Choose a reason for hiding this comment

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

Too verbose comment in code, can just say returns json for invoice entity and link to existing docs containing this sample.

@jitendra-1217
Copy link
Contributor

Merging this now.
Will fix formatting and other things in readme in a separate branch myself.

Thanks a lot @varunajmera0 👍

@jitendra-1217 jitendra-1217 merged commit f540d2a into razorpay:master Mar 11, 2019
@amitmahbubani
Copy link
Contributor

Thanks @varunajmera0 💯

@varunajmera0
Copy link
Contributor Author

thanks, @jitendra-1217 @amitmahbubani @captn3m0

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.

4 participants