Skip to content

[14.0][ADD] purchase_report_date_planned#215

Open
mathieudelva wants to merge 4 commits into
14.0from
14.0-purchase_report_date_planned
Open

[14.0][ADD] purchase_report_date_planned#215
mathieudelva wants to merge 4 commits into
14.0from
14.0-purchase_report_date_planned

Conversation

@mathieudelva

Copy link
Copy Markdown
Member

No description provided.

class PurchaseReport(models.Model):
_inherit = "purchase.report"

date_planned = fields.Datetime(compute="_compute_date_planned", store=True)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you do not have to create a computed field. it's just a normal field and as the model is based on a sql view the data will be here (with the override of _select and _group_by)

@sebastienbeau sebastienbeau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

minor change
Can you move this in oca : https://github.com/akretion/purchase-workflow ?

@mathieudelva

Copy link
Copy Markdown
Member Author

@sebastienbeau no because the module depends on purchase_usability

Comment thread purchase_report_date_planned/models/purchase_report.py Outdated
@sebastienbeau

Copy link
Copy Markdown
Member

@sebastienbeau no because the module depends on purchase_usability

Indeed I didn't see that the tree wview was added by odoo-usability.

Comment thread purchase_report_date_planned/views/purchase_report.xml
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.

3 participants