-
-
Notifications
You must be signed in to change notification settings - Fork 366
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[16.0][ADD] account_analytic_document_date: New module account_analyt… #694
base: 16.0
Are you sure you want to change the base?
[16.0][ADD] account_analytic_document_date: New module account_analyt… #694
Conversation
2e75749
to
6066e28
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, code review and tested in runboat
6066e28
to
a8f05bb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, tested in runboat
331cb37
to
73bc5d2
Compare
73bc5d2
to
edd9720
Compare
@pedrobaeza Could you review please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand the goal of having another date. I would understand that you want a different date for the analytic line, but not having dual dates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be ca.po
This module adds an other date to the account.move for analytics proposits
Usage:
Set Document Date on Invoice:
Analytic Line Generation:
Setting Document Date in Reconcile View:
cc https://github.com/APSL 160768
@miquelalzanillas @lbarry-apsl @javierobcn @peluko00 @ppyczko please review