[16.0][IMP] mis_builder: allow to export multiple report in one xls#669
[16.0][IMP] mis_builder: allow to export multiple report in one xls#669
Conversation
|
Hi @sbidoul, |
650a65d to
3392ff4
Compare
|
This looks good at first sight. If we don't have test coverage for this, it might be a good occasion to add some. |
3392ff4 to
e81811a
Compare
There's already a test that checks that it is possible to generate a xlsx report. I've added a small one for generation of a report over 2 instances. |
e81811a to
ce944fd
Compare
ce944fd to
8e6ee05
Compare
8e6ee05 to
ba28346
Compare
ba28346 to
6265544
Compare
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
|
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
|
@sbidoul, I'll soon port this PR in 19. Do you prefer to try have some review here to have it merged then, migrate to 19 or close this PR and open a new one in 19? |
Description
Allow to export multiple reports at once.
In one '.xls' where every report is a sheet.
To do so, I added one action 'Export XLS' visible on tree view of
mis_report_instance.