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

Report Category including all sub-categories #4789

Closed
whalley opened this issue Jul 6, 2022 · 6 comments
Closed

Report Category including all sub-categories #4789

whalley opened this issue Jul 6, 2022 · 6 comments
Assignees
Milestone

Comments

@whalley
Copy link
Member

whalley commented Jul 6, 2022

See: https://forum.moneymanagerex.org/viewtopic.php?t=10626

Hi,
I updated the application but there is no longer the possibility to extract the report including subcategories.

before there was the flag to indicate to take into account all subcategories

image

@whalley whalley added the bug label Jul 6, 2022
@whalley whalley added this to the v1.5.17 milestone Jul 6, 2022
@whalley whalley added question and removed bug labels Jul 6, 2022
@whalley
Copy link
Member Author

whalley commented Jul 6, 2022

This is where the use of regular expressions is needed. It is not very obvious to users. Not sure how we make this obvious. As @renato-mmex points out many times, we don't have a user guide that is kept up-to-date.

@whalley
Copy link
Member Author

whalley commented Jul 6, 2022

#4618

@vomikan
Copy link
Member

vomikan commented Jul 6, 2022

We can return the checkbox as it was before. If the checkbox is checked then add .* to the search text, if this line corresponds to a category from the list (Otherwise, the query string can be considered a regular expression.).
In principle, it is not a complicated fix.

I actually deleted this element in order to recreate it again, but with a new logic of work.

@whalley
Copy link
Member Author

whalley commented Jul 6, 2022

I like the idea....

  • If user selects a category (with no subcategory) we enable the checkbox
  • If they select the checkbox then the search, as you say, appends .*

@whalley whalley self-assigned this Jul 6, 2022
@renato-mmex
Copy link

I would also welcome this checkbox back.
Lately I have the impression that the usability of MMEX suffers because of the not self-explanatory new options.

whalley pushed a commit to whalley/moneymanagerex that referenced this issue Jul 6, 2022
@whalley whalley added fixed and removed question labels Jul 6, 2022
vomikan added a commit that referenced this issue Jul 6, 2022
fix(#4789): checkbox for including sub categories
@vomikan
Copy link
Member

vomikan commented Jul 6, 2022

It's definitely better than it was!

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

No branches or pull requests

3 participants