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

Sign of transfer from savings account #3382

Closed
McOrion opened this issue May 22, 2020 · 4 comments
Closed

Sign of transfer from savings account #3382

McOrion opened this issue May 22, 2020 · 4 comments
Labels
enhancement Requests for enhancements of existing stuff. fixed Bugs that are fixed (in a coming release).

Comments

@McOrion
Copy link

McOrion commented May 22, 2020

Bug description
I am running Firefly III version 5.2.5. and my problem is:

When making a transfer, initiated from the checking account on the dashboard, from the savings account to the checking account, the sign of the transfer is positive (green), both on the checkings account as on the savings account. I expected a positive (green) sign on the checking account, as the money is added there, but I expected a negative (red) sign on the savings account, as the money is withdrawn from this account.

Steps to reproduce
On the dashboard, create new transfer from the checking account.

Extra info
It's also working like this on the demo website. See transaction "withdraw money from savings" of $ 256,-

Bonus points

  • I searched and nobody reported this bug before---YES

  • I have added a stack trace from my log files---NO

  • I have added a screenshot.---YES
    Knipsel

  • I was able to replicate it on the demo site https://demo.firefly-iii.org/ ---YES

@JC5
Copy link
Member

JC5 commented May 22, 2020

Indeed. This is on purpose. Transfers are blue and positive on both ends. I’ll see if I can make them negative for the right account but they will stay blue.

@McOrion
Copy link
Author

McOrion commented May 22, 2020

At least it works as designed ;-)

Looking close to my screen, I agree with you on the blue color. Although I'm not aware of having difficulties seeing colors, I didn't noticed the blue color earlier. That would be OK for me, as it makes clear the transaction is a transfer.

On my personal savings account I'm only able to transfer money in and out from my checkings account. So all transactions have the blue color and have a positive sign. That makes it difficult to see which transaction adds to the account and which do not. It would be great to have a sign, but a workaround would mean adding some keyword in the description.

@JC5 JC5 added the enhancement Requests for enhancements of existing stuff. label May 23, 2020
JC5 added a commit that referenced this issue Aug 5, 2020
@JC5
Copy link
Member

JC5 commented Aug 5, 2020

Sometimes you have to sacrifice stuff to solve it. I dumped the "grouped" view, making the code a lot simpler.

The view (.twig) is a lot more complex, granted, but this works fine. Fixed in the next release.

All credits to @sephrat for fixing the hard part!

@JC5 JC5 added the fixed Bugs that are fixed (in a coming release). label Aug 5, 2020
@JC5 JC5 closed this as completed Aug 14, 2020
@github-actions
Copy link
Contributor

github-actions bot commented May 2, 2021

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 2, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests for enhancements of existing stuff. fixed Bugs that are fixed (in a coming release).
Projects
None yet
Development

No branches or pull requests

2 participants