Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parse error on ~ Monthly and aliases #28

Open
jorgb opened this issue Jan 21, 2022 · 3 comments
Open

Parse error on ~ Monthly and aliases #28

jorgb opened this issue Jan 21, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@jorgb
Copy link

jorgb commented Jan 21, 2022

Describe the bug
I use ledger-cli and came across this plugin. When using it, I noticed it cannot parse my ledger file/

To Reproduce
Steps to reproduce the behavior:

  1. Create a ledger file with stated text below
  2. Run the plugin on the ledger
  3. There be parse errors
  4. While ledger-cli works

Expected behavior
If this is a rewrite of ledger-cli, maybe skip the Monthly? Aliases are a must though. They should be supported even in text expansion. When I type b:Shopping it should understand I mean Household:Budget:Shopping (for example). And maybe present the aliases as valid autocomplete after scraping the ledger file, as it "seen" the entry before it can autocomplete the to / from fields with them.

Additional context
This does not parse;

; =========================================
; =   BUDGET 2022
; =========================================

alias a=Budget:Auto
alias b=Budget
alias u=Budget
alias chk=Accounts:Household
alias r=Accounts
alias w=Budget:Living
alias z=Budget:Pocket Money
alias s=Reserve

~ Monthly
    Budget:Shopping			$-xxx.00
    Budget:Care			$-xxx.00
    Budget:Clothes				$-xxx.00
    Budget:Pets			$-xxx.00
@jorgb jorgb added the bug Something isn't working label Jan 21, 2022
@tgrosinger
Copy link
Owner

@jorgb this is a complete rewrite, as I wanted the features to be available on all platforms, even ones that cannot use ledger-cli easily such as mobile. There absolutely are features of ledger-cli which are not yet implemented.

Aliases are mostly implemented, however there are probably some quirks that need to be worked out. In my recent rewrite of the transaction edit screen I did break its understanding of aliases, however they will still be parsed correctly when building the charts in the dashboard.

One question I have not fully come to a conclusion on: If you enter a transaction using an alias, what do you expect to be saved in the ledger file? Should it be saved exactly as written? Or should the alias be expanded before saving?

@tgrosinger
Copy link
Owner

Please also feel free to point me towards info about what the Monthly feature is. I'm not familiar with that one, which is part of the reason the plugin does not support it 🙂

@leonn3k
Copy link

leonn3k commented Dec 6, 2022

Periodic/recurring expenses is exactly what I was hoping for from this plugin, would love to see it implemented!

Here's some links to what jorgb was referring to:

My goal with using these functions of ledger is to be able to see what my daily/weekly/monthly expenses add up to compared to how much of an income I have so I can have more of an idea of how much money I have available to spend each week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants