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

Unable to record cash-in transaction #228

Closed
xentac opened this issue Feb 4, 2024 · 1 comment · Fixed by #229
Closed

Unable to record cash-in transaction #228

xentac opened this issue Feb 4, 2024 · 1 comment · Fixed by #229
Assignees
Labels
bug Something isn't working

Comments

@xentac
Copy link

xentac commented Feb 4, 2024

Describe the bug
I had someone give me money and I wanted to record it in beancount. I wasn't able to figure out how to add Cash to my wallet instead of remove it.

To Reproduce
Steps to reproduce the behavior:

  1. Create a transaction, with an amount of 10, from Assets:Cash to some Expense account. The transaction will be -10 Units.
  2. Create a transaction, with an amount of -10, from Assets:Cash to some Expense account. The transaction will be -10 Units.

Expected behavior
A clear and concise description of what you expected to happen.
One of those two options should let me add money to Cash. One might consider it an accounts:to, but then I'm just listing all my accounts in both accounts:from and accounts:to

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

  • OS: [e.g. iOS]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6]
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context
Add any other context about the problem here.

@xentac xentac added the bug Something isn't working label Feb 4, 2024
LucaBernstein added a commit that referenced this issue Feb 4, 2024
LucaBernstein added a commit that referenced this issue Feb 4, 2024
@LucaBernstein
Copy link
Owner

Hi xentac, thank you for your issue!

I checked on your report and I see the problem.

I will change the logic accordingly, as I cannot see any blockers not to do so (changes a bit the current behaviour, but should be manageable).

PR: #229

@LucaBernstein LucaBernstein linked a pull request Feb 4, 2024 that will close this issue
5 tasks
LucaBernstein added a commit that referenced this issue Feb 4, 2024
* Allow negative values to invert transaction

Solves #228

* Update Go dependencies
@LucaBernstein LucaBernstein self-assigned this Feb 4, 2024
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

Successfully merging a pull request may close this issue.

2 participants