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][IMP] epa_account_custom: add flexible installments #64

Merged

Conversation

WesleyOliveira98
Copy link
Contributor

Ticket HT00721

cc: @marcelsavegnago @kaynnan @Matthwhy

@WesleyOliveira98 WesleyOliveira98 force-pushed the 14.0-imp-epa_account_custom-flexible_installments branch 3 times, most recently from 0e3a6f8 to 661134f Compare June 17, 2024 14:52
@WesleyOliveira98 WesleyOliveira98 marked this pull request as ready for review June 17, 2024 15:02
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

@WesleyOliveira98 WesleyOliveira98 force-pushed the 14.0-imp-epa_account_custom-flexible_installments branch from 661134f to da8e6b7 Compare June 17, 2024 18:07
@WesleyOliveira98
Copy link
Contributor Author

WesleyOliveira98 commented Jun 17, 2024

add attrs="{'invisible': [('invoice_payment_term_id', '=', False)]}" to the field

@WesleyOliveira98 WesleyOliveira98 force-pushed the 14.0-imp-epa_account_custom-flexible_installments branch from da8e6b7 to 166e465 Compare June 17, 2024 18:30
@marcelsavegnago
Copy link
Contributor

@WesleyOliveira98 podemos seguir com o merge ?

@WesleyOliveira98
Copy link
Contributor Author

@WesleyOliveira98 podemos seguir com o merge ?

@marcelsavegnago podemos sim!

@marcelsavegnago marcelsavegnago merged commit 0911d13 into 14.0 Jun 19, 2024
2 of 4 checks passed
@marcelsavegnago marcelsavegnago deleted the 14.0-imp-epa_account_custom-flexible_installments branch June 19, 2024 15:41
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.

None yet

3 participants