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

[Bug]: In/Out column cannot be selected #4571

Open
1 task done
Gmanicus opened this issue Mar 8, 2025 · 8 comments · May be fixed by #4715
Open
1 task done

[Bug]: In/Out column cannot be selected #4571

Gmanicus opened this issue Mar 8, 2025 · 8 comments · May be fixed by #4715
Labels
bug Something isn't working needs info We need more information from the OP before continuing

Comments

@Gmanicus
Copy link

Gmanicus commented Mar 8, 2025

Verified issue does not already exist?

  • I have searched and found no existing issue

What happened?

When importing transactions from a CSV, I appear to be unable to specify which column determines whether the amount is in/outflow.

I see "In/Out" in the preview, but no option to select which column has the value.

Image

How can we reproduce the issue?

  1. Import CSV with "Split into separate inflow outflow columns" enabled
  2. Import CSV again, but disable split mode
  3. Check "Select column to specify if amount goes in/out"
    In-Out-Example.csv

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Chrome, Other

Operating System

Windows 10

@Gmanicus Gmanicus added the bug Something isn't working label Mar 8, 2025
@youngcw
Copy link
Member

youngcw commented Mar 8, 2025

Have you typed in the column to use in the field next to the checkbox?

@Gmanicus
Copy link
Author

Gmanicus commented Mar 8, 2025

I believe so. Let me double-check. Yes. The column name is "Credit or Debit". Entering that doesn't update the preview with the correct in/out flows. If I understand correctly, I think that text box is for determining which value within the column is "positive" and by exclusion which is "negative" as well.

@youngcw
Copy link
Member

youngcw commented Mar 8, 2025

Seems just fine to me

Image

@Gmanicus
Copy link
Author

Gmanicus commented Mar 8, 2025

Huh. Something's odd on my end. When I check "Select in/out column", it auto-checks "Split amount into separate inflow/outflow columns" and locks me out of disabling them. Now I can't change either option even if I exit out of the modal. If I come back, they're still checked 👀

I wonder if this is a bug caused by having an account that already has inflow and outflow columns and saved transactions.

Image

@Gmanicus
Copy link
Author

Gmanicus commented Mar 8, 2025

Tried setting up a dev environment but couldn't get it started up. Will probably take another crack at it tomorrow to see if I can take a peek at the code. Thanks for taking a look and troubleshooting with me 🙏

@MatissJanis
Copy link
Member

@Gmanicus how's this looking? All good?

I'd like to close the issue if all good.

@MatissJanis MatissJanis added the needs info We need more information from the OP before continuing label Mar 28, 2025
@Gmanicus
Copy link
Author

Gmanicus commented Mar 29, 2025

Hey @MatissJanis, sorry for the delay on this. Got busy over the last few weeks and never got back to this.

Still stuck, unfortunately. Going to get back on that dev environment right now and will report back.

EDIT: Finally figured out what was going wrong and got the dev container up. I'll start poking around.

@Gmanicus
Copy link
Author

Gmanicus commented Mar 29, 2025

I wonder if this is a bug caused by having an account that already has inflow and outflow columns and saved transactions.

I think I was spot on here.

When the In/Out status is changed, it reloads the preview. This then reloads the transactions, as well as the Split amount status. Split amount appears to be enabled if either the Inflow or Outflow columns have values.

Additionally, In/Out appears to be designed to be used in replacement of Split amount. So, if In/Out is enabled, Split amount is disabled, and vice versa.

Due to my configuration, I have a bit of a catch 22 situation. My transactions already have Inflow/Outflow columns, so Split amount is enabled. My configuration has also saved In/Out as enabled since I tried to set it previously. So, both are enabled, making me unable to disable either of them.

If I work around this by changing the disabled condition, Split amount still remains checked since I already have Inflow/Outflow columns, making me unable to set up In/Out. How I got there in the first place, I do not know.

I have some plan of how to fix this:

  1. The In/Out behavior should work with Split amount, not against it, and change how the transaction value is saved accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs info We need more information from the OP before continuing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants