Skip to content
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

[12.0] l10n_br_account inherits aml #1205

Closed
wants to merge 4 commits into from

Conversation

mileo
Copy link
Member

@mileo mileo commented Mar 13, 2021

Depends of #1125

@mileo
Copy link
Member Author

mileo commented Mar 13, 2021

Hi Guys,

This concept is the sync of aml with fiscal payment of #1125. There are other ways to do that, like as done in the old #806

Can we discuss the best way to do it?

We propose the #820 a year ago, can you review it?

The only fresh feed back is #849 (review)

Of course we can remove the account.payment.term copy, the user will lost some usability (only manual lines), but this can be done in a separate module.

@github-actions
Copy link

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 github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Apr 16, 2023
@github-actions github-actions bot closed this May 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12.0 needs review stale PR/Issue without recent activity, it'll be soon closed automatically. work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant