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

add the option to turn off item account filtering #778

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jkroso
Copy link

@jkroso jkroso commented Nov 29, 2023

I'm not sure this is good accounting practice but I prefer to be able to record asset purchases the same way I record expenses.

@mildred
Copy link
Contributor

mildred commented Dec 2, 2023

Basically, you're adding a toggle to allow selecting any kind of account for an item. Would it be possible to have a behavior that would work for you without introducing a new toggle? What kind of account are you using that is today filtered out?

@18alantom what do you think?

@jkroso
Copy link
Author

jkroso commented Dec 2, 2023

Well I'm trying to assign an item to my "tools" account which is a fixed asset account. So yeah I could just change the default filtering instead if you think that would work for everyone.

@mildred
Copy link
Contributor

mildred commented Dec 2, 2023

Perhaps a better option would be to keep the account filtering but let the user enter accounts outside of the filter if he really wants to.

@jkroso
Copy link
Author

jkroso commented Dec 3, 2023

So add a button in the drop down to turn off filtering in a specific case rather than globally?

@18alantom
Copy link
Member

@mildred I think since this is a change that can lead to undefined behavior (causing issues by confused users later on) it should be restricted in someway. Perhaps behind a feature flag.

The feature itself could be an option in the drop down that allows turning of filtering (such as what @jkroso suggested).

If not feature flag, the option could be made visible if the list is empty for a given query.

@mildred
Copy link
Contributor

mildred commented Dec 5, 2023

@18alantom so, do you think the global option introduced in this PR is the good way to do it or should it specifically be in the feature flag section of the options?

@18alantom
Copy link
Member

@mildred I think, ideally the feature flag should be to enable optionally removing filters when entering Link field values (i.e. across all Link fields).

The feature flag in this PR completely switches it off for two fields. Disabling filters for a Link field seems like an infrequent requirement.

Also, there have been instances wherein users have needed filters across other fields to be removed. Introducing individual feature flags for each would unnecessarily populate the feature flag section.

@mildred
Copy link
Contributor

mildred commented Nov 20, 2024

Implemented in #1022

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants