invoiceninja / sdk-php

PHP wrapper for Invoice Ninja's REST API
https://www.invoiceninja.com
83 stars 41 forks source link

integration of send invoice #49

Closed TheJenne18 closed 2 years ago

hillelcoren commented 4 years ago

Thanks for creating the PR!

Is there a reason you added a new class rather than add the function to the invoice model?

TheJenne18 commented 4 years ago

To stay consistent with the routing: public static $route = 'email_invoice';

hillelcoren commented 4 years ago

Hmm... I think it may be better to add the method to the existing class and use the 'email_invoice' option

https://invoice-ninja.readthedocs.io/en/latest/api.html#options