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]: Suggested Payees is inconsistent #3316

Closed
2 tasks done
psybers opened this issue Aug 25, 2024 · 5 comments · Fixed by #3318
Closed
2 tasks done

[Bug]: Suggested Payees is inconsistent #3316

psybers opened this issue Aug 25, 2024 · 5 comments · Fixed by #3318
Labels
bug Something isn't working

Comments

@psybers
Copy link
Contributor

psybers commented Aug 25, 2024

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

If you view the suggested payees, after favoriting two, you might see something like this:

image

But if you then go to add a new transaction, the payee list looks like this:

image

It seems to forget to show the favorites.

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Chrome

Operating System

Mac OSX

@psybers psybers added the bug Something isn't working label Aug 25, 2024
@psybers
Copy link
Contributor Author

psybers commented Aug 25, 2024

Related to #2814.

@qedi-r might want to take a look.

@qedi-r
Copy link
Contributor

qedi-r commented Aug 25, 2024

This seems to be limited to the case where you are selecting the payee without an account selected. Investigating what is different in this path.

@qedi-r
Copy link
Contributor

qedi-r commented Aug 25, 2024

Ah, I guess it's time to fix this then: #2814 (comment)

@psybers
Copy link
Contributor Author

psybers commented Aug 25, 2024

Ah, I guess it's time to fix this then: #2814 (comment)

I'm not entirely sure the two are related, but maybe. Either way I guess we can try to fix that here.

@qedi-r
Copy link
Contributor

qedi-r commented Aug 25, 2024

My assumption here is that not displaying any payees when an account is not yet selected on new transactions is the bug as shown below:

image

if this is fixed, then the missing favorites is also fixed. The fix is the payees.filter line 1243 on TransactionTable.tsx.

If this is intended to not display payees, then we need to know when this is the case, and not add common payees to this list.

@youngcw youngcw linked a pull request Aug 26, 2024 that will close this issue
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