search

fastruby / harvesting

Ruby wrapper for the Harvest API v2
MIT License
28 stars 30 forks source link

Line Item Models #36

Open jonesabby opened 5 years ago

jonesabby commented 5 years ago

Currently unable to create invoices without the Line Item model.

etagwerker commented 5 years ago

@jonesabby Interesting. I haven't tried it myself yet. What error do you get?

jonesabby commented 5 years ago

Sorry. Actually no error - the Line Item model just doesn't exist to create an invoice. I see there is a PR for it from February - but doesn't look like any movement on that.

On Tue, Apr 16, 2019 at 10:23 AM Ernesto Tagwerker notifications@github.com wrote:

@jonesabby https://github.com/jonesabby Interesting. I haven't tried it myself yet. What error do you get?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ombulabs/harvesting/issues/36#issuecomment-483709018, or mute the thread https://github.com/notifications/unsubscribe-auth/AC9RUAbetKvmjfZ11vk1Q9N0zmurBl08ks5vherygaJpZM4cyfAV .

-- Abby Jones | 515.440.1242 Entrepreneurial Technologies Inc. 2655 86th Street | Urbandale, Iowa 50322 abby@entretechno.com | www.entretechno.com

etagwerker commented 5 years ago

@jonesabby Does #35 solve this issue for you?

jonesabby commented 5 years ago

Yes - it looks like that will do the trick when it is merged.

On Tue, Apr 16, 2019 at 10:42 AM Ernesto Tagwerker notifications@github.com wrote:

@jonesabby https://github.com/jonesabby Does #35 https://github.com/ombulabs/harvesting/pull/35 solve this issue for you?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ombulabs/harvesting/issues/36#issuecomment-483717023, or mute the thread https://github.com/notifications/unsubscribe-auth/AC9RUB4sBUyIabiHBLzLMcDmD9nlxfuyks5vhe9pgaJpZM4cyfAV .

-- Abby Jones | 515.440.1242 Entrepreneurial Technologies Inc. 2655 86th Street | Urbandale, Iowa 50322 abby@entretechno.com | www.entretechno.com

etagwerker commented 4 years ago

@jonesabby It seems that that PR was never merged. Feel free to take a stab at it.