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

[IMP] Add payment terms to purchase order template #159027

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

andreasmuck
Copy link

@andreasmuck andreasmuck commented Mar 23, 2024

Description of the issue/feature this PR addresses:
Payment terms are an important attribute of purchase orders and the default template does not include them.

Current behavior before PR:
Default purchase order document template does not include payment terms

Desired behavior after PR is merged:
Payment terms are shown on document header

CLA signature
@robodoo
Copy link
Contributor

robodoo commented Mar 23, 2024

Pull request status dashboard.

@C3POdoo C3POdoo requested a review from a team March 23, 2024 20:28
@andreasmuck andreasmuck changed the title Sign andreasmuck.md [CLA] Sign andreasmuck.md Mar 23, 2024
@C3POdoo C3POdoo added the CLA Contributor License Agreement label Mar 23, 2024
@C3POdoo C3POdoo requested a review from a team March 23, 2024 23:14
@mart-e
Copy link
Contributor

mart-e commented Mar 25, 2024

Hi @andreasmuck

I see you have included as well a change in purchase. Is it intended or you just want to merge your CLA here ?
Can you detail your change so the @odoo/rd-logistics can review it ?

@andreasmuck
Copy link
Author

Hi @mart-e

I included the change detail.

Regards

@andreasmuck andreasmuck changed the title [CLA] Sign andreasmuck.md [IMP] Add payment terms to purchase order template Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants