Kianosh Pourian kianoshp

Organizations

@vmware
kianoshp commented on pull request kianoshp/nvoice#34
@kianoshp

Let's continue with this idea and expand on it. How will we communicate to the consumer of the API that an error has occurred? We need to establish…

kianoshp commented on pull request kianoshp/nvoice#35
@kianoshp

We should be setting the invoice id based on the invoice that we will be creating for the test. This hard coding of the invoiceId will not allow fo…

@kianoshp
kianoshp merged pull request kianoshp/nvoice#33
@kianoshp
added a swagger.json file with companies
9 commits with 7,924 additions and 22,055 deletions
@kianoshp
  • @kianoshp a53bf62
    Changed all item reference to invoiceItem
@kianoshp
  • @kianoshp b45ee0d
    Fixed the update and delete issue
@kianoshp
@kianoshp
  • @kianoshp 1e55231
    Fixed some of the swagger issues and got it work
@kianoshp
  • @kianoshp 50bf3e4
    I am able to retrieve the inidividual invoice-item, now on to updatin…
@kianoshp
@kianoshp
  • @kianoshp 85a63b7
    added the existing site to the repo
kianoshp created repository kianoshp/family-eye-care
@kianoshp
kianoshp merged pull request kianoshp/nvoice#31
@kianoshp
Invoice item
22 commits with 560 additions and 1 deletion
kianoshp commented on pull request kianoshp/nvoice#31
@kianoshp

How can we rely on that? Also didn't we already delete all the invoices that we created in the unit test?

kianoshp commented on pull request kianoshp/nvoice#31
@kianoshp

How is this create on invoiceItem going to work if we haven't created an invoice yet?

kianoshp commented on pull request kianoshp/nvoice#31
@kianoshp

let's change all the item references to invoiceItem so it will be more descriptive.

@kianoshp
  • @kianoshp 1b1f97c
    Added a solution for the READ option of the invoice items
kianoshp created branch new-site at kianoshp/the-site
kianoshp commented on pull request kianoshp/nvoice#31
@kianoshp

The CRUD in this process will work a little bit different. All invoice-items need an invoice to be attached to and they are not able to exist on th…

@kianoshp
kianoshp merged pull request kianoshp/nvoice#30
@kianoshp
Invoice api
3 commits with 260 additions and 0 deletions
kianoshp commented on pull request kianoshp/nvoice#30
@kianoshp

This looks good :+1:

@kianoshp
kianoshp merged pull request kianoshp/nvoice#27
@kianoshp
User crud
20 commits with 254 additions and 1 deletion
kianoshp commented on pull request kianoshp/nvoice#27
@kianoshp

:100:

kianoshp commented on pull request kianoshp/nvoice#27
@kianoshp

The unit tests are not passing, can you take a look at them?