-
Notifications
You must be signed in to change notification settings - Fork 184
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
TO BE DELETED: One category list: nested category #1552
TO BE DELETED: One category list: nested category #1552
Conversation
…com/moneymanagerex/general-reports/tree/master/packages/Category/OneCategoryList). Main Change: * add one_category_list.sql * Alter query_alldata.sql for using recursive category * alter query_categorysubcategory.sql for using recursive category * alter query_mobiledata.sql for using recursive category * adapt CategoriesReportFragment * adapt CategoryRepository * adapt CategoryService * adapt ViewMobileData Tested-by: wolfsolver Test execution: * Category List * Category Edit * Category Add - Root * Category add - Child * Category Delete * Report Category * Report Income/Expence * Search view * Transaction List * Transaction Edit * Transaction Add
Dont' know how to fix failed check on Codacy... @guanlisheng any suggestion? |
as the change is big, it is better to split it into multi-small pieces and do enough tests (especially on compatibility) |
@guanlisheng can you help me to understand how to change SQL query for codacy compatibility? https://app.codacy.com/gh/moneymanagerex/android-money-manager-ex/pullRequest?prid=13555377 |
Does make sense to create a beta release for this? |
if you are confident, let's roll out directly. don't forget to document related changes |
Initial release for recursive categories
See #1376 (comment)