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]: deleted payees still show in suggested list #3317

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

[Bug]: deleted payees still show in suggested list #3317

psybers opened this issue Aug 25, 2024 · 3 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 have a commonly occurring payee and then you delete it, it will still show in the list of suggested payees.

For example here, I actually deleted "Kroger" from my payees list (all transactions now show blank payee):

image image

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.

@psybers
Copy link
Contributor Author

psybers commented Aug 25, 2024

Probably the query needs to filter with p.tombstone = 0, but in my test it looks like there is a caching issue with that in place and it will not work until you refresh the page.

@qedi-r
Copy link
Contributor

qedi-r commented Aug 25, 2024

Yeah. We'd need some way to control the cache on the accounts page. It looks like both just the normal payee list and the common payee list cache the result until the page is reloaded.

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