Add col_21 -> col_30 to T_Report

Activity

Show:

Nicolas Micoud November 25, 2019 at 5:41 AM

Hi ,

Oops...
Patch updated (with migration scripts and based on v7.1).
But not tested on 7.1 as I do not have a development environnment for this version ATM.
As the change on FinReport is just SQL, there should not be any issue (as it was working on 6.2)

Thanks,

Nicolas

Carlos Ruiz November 21, 2019 at 7:48 PM

Hi , the patch shows conflicts to be applied in the default branch.

Also, the migration scripts are not included in the patch.

Regards,

Carlos Ruiz

Norbert Bede November 21, 2019 at 5:30 PM

properly implemented json field should resolve this very generic way !

Nicolas Micoud November 21, 2019 at 4:35 PM

Hello ,

Please find a patch that adds several columns to T_Report.

nb: For testing, make sure to clean the PrintFormat field on the CreateReport process
nb2 : Could you consider integrating it before 7.1 (as it contains migration scripts) ?

Regards,

Nicolas

Fixed

Details

Assignee

Reporter

Labels

Fix versions

Priority

Created November 21, 2019 at 4:08 PM
Updated January 2, 2020 at 8:14 AM
Resolved November 27, 2019 at 9:09 AM