search

camelcasetechsd / testing-engine

1 stars 0 forks source link

Custom Report Join Table fields #335

Closed HeshamHassan closed 6 years ago

HeshamHassan commented 6 years ago

URL: http://skilledmatrix.com/admin/backend/reports/12/edit Path: Admin> Backend> Custom Report Description:

ahmedelbougha commented 6 years ago

@mahmoud-hussien1 Please make sure to make the report contains at least one field, but don't make it mandatory to select columns from all tables.

HeshamHassan commented 6 years ago

@ahmedelbougha Any update?

mahmoud-hussien1 commented 6 years ago

PR - https://bitbucket.org/camelcasetech/testing-engine/pull-requests/47/issue-335-create-custom-validation-rule/diff

ahmedelbougha commented 6 years ago

done, @HeshamHassan check and confirm.

HeshamHassan commented 6 years ago

@ahmedelbougha I cannot check. the table order in the report changes each time I change the checking factors.

HeshamHassan commented 6 years ago

@ahmedelbougha Any update?

ahmedelbougha commented 6 years ago

@HeshamHassan please create new custom report and check the functionality is working fine (create/edit/run) and if not please let me know.

ahmedelbougha commented 6 years ago

@HeshamHassan any update??!!!

ahmedelbougha commented 6 years ago

@HeshamHassan we added new field group to use it for ordering the table, check and confirm please. @HeshamHassan, @Ali-Galal This issue is duplicate of issue #337 please close and concentrate on one issue only.

HeshamHassan commented 6 years ago

@ahmedelbougha Kindly note: Issue #337 is about the used tables order in the report. Issue #335 is about the joining between tables and the mandatory fields. Each issue of them affects the report and prevents the running of the report.

ahmedelbougha commented 6 years ago

@HeshamHassan When is the last time did you try to create report? the functionality is working since some time and I didn't hear back from you about this. FYI, I'll remove all old reports to avoid confusion and please start to make new ones.

HeshamHassan commented 6 years ago

@ahmedelbougha Good Remove all the old confusion report. So, I can create new clear reports. Inform me when done to start my checking.

ahmedelbougha commented 6 years ago

@HeshamHassan old reports have been removed. Please note this issue has been opened according to this description:

We used a join table to join other tables together. But I don’t want to show its data in the report when it runs. The report creator enforces me to show at least one of its columns in the report.
We can check it together in report 12: Named “ Ali “

And this issue has been fixed since long time ago, so either close this issue and open another issue or stick to one issue to discuss all issues of Custom Reports.

Please confirm.

HeshamHassan commented 6 years ago

@ahmedelbougha I cannot deal as well with the custom report module. Kindly arrange an online appointment to fix all issues about that module.

ahmedelbougha commented 6 years ago

@HeshamHassan You mean you cannot use the module. If you cannot deal with the module, the close all related issues and create a new one of how to use. @Ali-Galal Would you please check and confirm what is needed?

Ali-Galal commented 6 years ago

@ahmedelbougha this issue has been settled @HeshamHassan close this issue @HeshamHassan close issue #337 @HeshamHassan open new issue contains all issues related to the custom report in checkbox items