[17.0][MIG] project consumable #1569
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
/ocabot migration project_consumable |
| ) | ||
|
|
||
| @api.onchange("type") | ||
| def _onchange_type(self): |
There was a problem hiding this comment.
Make project_ok fields compute/stored/readonly=False to avoid onchanges.
| class Project(models.Model): | ||
| _inherit = "project.project" | ||
|
|
||
| company_currency_id = fields.Many2one( |
There was a problem hiding this comment.
Is this at the right place (this module ?) ?
Contributor
Author
There was a problem hiding this comment.
I admit I can't remember why I'm not using the existing currency_id field in the monitory field... I've moved to the existing currency_id field
Thanks !
petrus-v
force-pushed
the
17.0-project_consumable
branch
from
73b34f1 to
929e591
Compare
In the previous implementation project_id field was reused on account move line but a lot of the odoo code source assume that an account move line with a project_id is a timesheet, making very hard to distinguish Materials and Timesheet. Adding a consumable_project_id it avoid to breaks existing code make module much more easier to maintains
petrus-v
force-pushed
the
17.0-project_consumable
branch
from
929e591 to
ed448e3
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Migrate project consumable #1410 from v14.0
with main differences: