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

[14.0][ADD] epa_account_manager: add new module #53

Merged
merged 1 commit into from
May 2, 2024

Conversation

Matthwhy
Copy link
Contributor

@Matthwhy Matthwhy commented Apr 26, 2024

Esse módulo coloca o manager da conta analítica na fatura. A débora preferiu ocultar o campo team_id, e no lugar dele colocar o manager_id

Copy link
Contributor

@kaynnan kaynnan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@WesleyOliveira98 WesleyOliveira98 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Matthwhy só incluir a versão no nome da PR, exemplo: [14.0][ADD]

@marcelsavegnago marcelsavegnago changed the title [ADD] epa_account_manager: add new module [14.0][ADD] epa_account_manager: add new module Apr 29, 2024

analytic_accounts = record.invoice_line_ids.mapped("analytic_account_id")
if analytic_accounts:
record.manager_id = analytic_accounts[0].manager_id.id
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vale um or False para evitar problema ?

@Matthwhy Matthwhy force-pushed the 14.0-add-epa_account_manager branch 5 times, most recently from 9295a49 to dfa203b Compare May 2, 2024 18:47
@Matthwhy Matthwhy force-pushed the 14.0-add-epa_account_manager branch from dfa203b to 19d5655 Compare May 2, 2024 19:56
@Matthwhy Matthwhy force-pushed the 14.0-add-epa_account_manager branch from 19d5655 to 6ec200d Compare May 2, 2024 20:06
@marcelsavegnago marcelsavegnago merged commit 107eafa into 14.0 May 2, 2024
2 of 4 checks passed
@marcelsavegnago marcelsavegnago deleted the 14.0-add-epa_account_manager branch May 2, 2024 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants