Skip to content
This repository has been archived by the owner on Jan 7, 2021. It is now read-only.

Enhancements #1

Open
muenzel opened this issue Jun 18, 2018 · 2 comments
Open

Enhancements #1

muenzel opened this issue Jun 18, 2018 · 2 comments

Comments

@muenzel
Copy link

muenzel commented Jun 18, 2018

Thank you for this library. I would prefer this one over medici because I think the use of a relational database is more sane for accounting purposes than using a NoSQL DB.

But there is one drawback with this library which makes it not useable for my purposes: I can't store and query metadata with the journal entries like with medici.

Looking at your example makes no sense to me:
myBook.balance({ account:'Assets:Accounts Receivable' }).then((balance) => { console.log("Joe Blow owes me", balance.total); });

When I look this up in medici, it makes sense again:

  1. In the journal entry, I can store metadata like the client
    .credit('Income', 1000, {client: 'Joe Blow'})
  2. And then I can use the metadata for the queries
    myBook.balance({ account:'Assets:Accounts Receivable', client:'Joe Blow' }).then((balance) => { console.log("Joe Blow owes me", balance); });

Another thing is the REST service. I really like the idea of having a kind of microservice for this purpose. But the drawback here: In the REST service I can't do everything which I can do with the API, e.g. Voiding Transactions.

Thanks again for this project.

@muenzel
Copy link
Author

muenzel commented Jul 1, 2018

Any thoughts on this?

@CjS77
Copy link
Owner

CjS77 commented Jul 5, 2018

Hi.. So unfortunately I don't have time to do any work on this project at the moment and probably won't have time for several months.

It looks like you have a good idea of how to add the feature you want. Why not implement it yourself and submit it as a PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants