stock-logistics-reporting icon indicating copy to clipboard operation
stock-logistics-reporting copied to clipboard
verified publisher icon OCA

[IMP] stock_average_daily_sale: change order of columns in the report

Open twalter-c2c opened this issue 9 months ago • 6 comments

After using the module for some time, we find the column order not most readable/logical. Here you can find a suggested order.

I added also a possibility to hide/show most of the columns.

twalter-c2c avatar Mar 18 '25 15:03 twalter-c2c

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

OCA-git-bot avatar Mar 25 '25 12:03 OCA-git-bot

I integrated the idea in the last version https://github.com/OCA/stock-logistics-reporting/pull/403/commits/d5add518e35110eb0acb1a854a24a0fc7766bfb4

jbaudoux avatar Apr 07 '25 16:04 jbaudoux

I wouldn't make the metrics hidden by default as it's all the purpose this report. I think this change is more based on a customer preference.

Yes, I agree. I made the metrics fields visible by default here.

twalter-c2c avatar Apr 08 '25 11:04 twalter-c2c

@jbaudoux have your concerns been addressed now?

twalter-c2c avatar Apr 09 '25 08:04 twalter-c2c

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. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Oct 12 '25 12:10 github-actions[bot]

Hello @jbaudoux can you take a look again and (hopefully) approve?

twalter-c2c avatar Oct 17 '25 09:10 twalter-c2c