-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
[ADD]pms: Self billing folios configuration #310
Conversation
d7b748c
to
9a4d542
Compare
/ocabot merge minor |
This PR looks fantastic, let's merge it! |
@DarioLodeiros your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-310-by-DarioLodeiros-bump-minor. After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red. |
/ocabot merge minor |
This PR looks fantastic, let's merge it! |
It looks like something changed on |
Congratulations, your PR was merged at 6bc2475. Thanks a lot for contributing to OCA. ❤️ |
Enhancements to the billing process by adding support for self-billed journals and taxes. The changes ensure that the system correctly applies self-billed journals and taxes when the partner is the same as the company partner.
Key changes include:
Self-billing support:
self_billed_journal_id
andself_billed_tax_ids
fields to theResCompany
model to store the self-billed journal and taxes information.Invoice and tax computation:
_get_folio_default_journal
method inpms_property.py
to return the self-billed journal if the partner is the company partner._compute_tax_ids
method inpms_reservation.py
to check for self-billed taxes when the partner is the company partner._compute_tax_ids
method inpms_service.py
to apply self-billed taxes when the service partner is the company partner.