Add capability to delete existing transaction entry

[utkuufuk]

@vitokhangnguyen This is a very good idea!

[Aravind22]

May I take a look into it? I am thinking of a scenario like this.
"User enters the description of the transaction he/she wants to delete, app displays last 5/10 transactions with the same description and user chooses which to delete".
If this seems fine, I can make it work.

[vitokhangnguyen]

Right now, the command budget log is displayimg the line index of an income or expense entry. User can leverage that index and use it to delete an entry. For eg...

budget delete income 3 aug

will delete the 3rd income entry of August spreadsheet.

[utkuufuk]

I think we could do both:

1. If the user runs the command:

   budget delete <income|expense> <index>

   We'll handle it as @vitokhangnguyen suggested. I think the command should not accept a month argument, because the user can only see the transactions for the current month when he/she runs budget log

2. If the user runs the command:
   budget delete <query>

   We could display search results as @Aravind22 suggested, with a generated ID for each transaction. Then wait for a second input for the ID. (This can be added to the edit operation as well.)

Personally I'd prefer the first option if I had to choose only one.

[vitokhangnguyen]

I think the month argument is ok since the user can see other month's transaction through

budget log <month>

and the result does show the line index.

However, we can do it your way and not having it for now. I just think it does not add much complexity to the problem.

[utkuufuk]

@vitokhangnguyen That's right, I honestly forgot about that feature 🤣

Then the command should default to the current month if only 3 arguments are provided. A particular month should be used if a 4th argument is provided, as in your example.

BTW, we could add that optional 4th argument to the edit command as well. I'll create a new issue for that in case you or anybody else wants to work on it.

[vitokhangnguyen]

@utkuufuk In the edit command, I thought about it and think that it does make it ambiguous. The edit command takes a transaction as an argument that has a date field. Currently, the program is choosing the spreadsheet id based on the month of that date field.

[utkuufuk]

@vitokhangnguyen Yeah that makes sense, let's ignore the optional month argument for the edit command for now.

[Aravind22]

I am facing an issue while using this app, i followed the steps and installed it successfully but when i invoke budget, i am getting command not found.

[utkuufuk]

@Aravind22 In your .bashrc or .zshrc, make sure that the correct directories are in your $PATH. For instance, here's my configuration:

export PATH="$HOME/.config/composer/vendor/bin:$HOME/.local/bin:$HOME/bin"

And the budget executable is located in:

~/.local/bin

which is in my $PATH