account-financial-reporting icon indicating copy to clipboard operation
account-financial-reporting copied to clipboard

Published 20 hours ago • verified publisher icon OCA

[13.0][ADD]account_move_line_report_xls

Open luc-demeyer opened this issue 4 years ago • 13 comments

luc-demeyer avatar Mar 25 '20 08:03 luc-demeyer

@luc-demeyer Wouldn't it be useful to add an option to export all journal items in the current filter? When you want to export an high amount of records you first would need to load them in your browser before you can export?

CasVissers-360ERP avatar Aug 17 '20 08:08 CasVissers-360ERP

@CasVissers-360ERP adding such an option would be easy to do but for this you can also use the general ledger export in the OCA account_financial_report hence I am not sure if it's worth the effort.

luc-demeyer avatar Aug 17 '20 08:08 luc-demeyer

@luc-demeyer The general ledger export layout is completely different (grouped by ledger account) and harder to use while building pivots etc.

CasVissers-360ERP avatar Aug 17 '20 08:08 CasVissers-360ERP

@luc-demeyer Any chance this option will be added?

CasVissers-360ERP avatar Sep 21 '20 05:09 CasVissers-360ERP

@CasVissers-360ERP I placed it on my todo list but not with highest prioriry. I also don't think we should wait on this new feature to merge this one. From an UI standpoint adding this option can be done in a several ways:

  • add two sidebar options (how should we call both to make it clear)
  • pop-up wizard allowing to select
  • config option
  • ... Any suggestion ?

luc-demeyer avatar Sep 21 '20 07:09 luc-demeyer

I think two sidebar options would be intrusive.

I prefer a configuration option, keeps the module (mega) user-friendly and avoids adding a wizard.

I agree it could be merged as-is.

CasVissers-360ERP avatar Sep 21 '20 07:09 CasVissers-360ERP

@luc-demeyer let me know if you need a review here.

fshah-initos avatar Mar 10 '21 05:03 fshah-initos

Can you squash out the 'greenify' commit?

StefanRijnhart avatar Jun 01 '21 11:06 StefanRijnhart

/ocabot rebase

rafaelbn avatar Apr 05 '22 07:04 rafaelbn

Congratulations, PR rebased to 13.0.

OCA-git-bot avatar Apr 05 '22 07:04 OCA-git-bot

Hello @luc-demeyer ! Please, could you squash and rebase to check if we can merge this one ?

rafaelbn avatar Apr 05 '22 07:04 rafaelbn

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 May 08 '22 18:05 OCA-git-bot

@luc-demeyer Can you squash your commits?

etobella avatar Oct 16 '22 20:10 etobella