Skip to content

Conversation

@glitchov
Copy link
Contributor

This modules adds a button on already invoiced timesheet sale order lines to update the unit price. It creates a new order line with the new price and re-links projects, tasks, milestones and so line employee maps from the old line to the new line.

@glitchov glitchov force-pushed the 19.0-add-sale_order_update_timesheet_price-hda branch from be599f8 to 54e2dec Compare December 16, 2025 13:09
Copy link

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

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

Some changes

@rousseldenis
Copy link

@glitchov

@glitchov glitchov force-pushed the 19.0-add-sale_order_update_timesheet_price-hda branch from 54e2dec to 2c4ec71 Compare December 17, 2025 13:39
@glitchov glitchov force-pushed the 19.0-add-sale_order_update_timesheet_price-hda branch from 2c4ec71 to 6f13797 Compare December 17, 2025 13:59
order_line_name = fields.Text(related="order_line_id.name", readonly=True)
old_unit_price = fields.Float(related="order_line_id.price_unit", readonly=True)
new_unit_price = fields.Float(required=True, digits="Product Price")
old_line_prefix = fields.Char(default="[Old Rate]")

Choose a reason for hiding this comment

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

Should be great to have a configuration parameter for this default value

Copy link
Member

Choose a reason for hiding this comment

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

This can come later if needed.

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@ivs-cetmix
Copy link
Member

Hey @glitchov thank you for your contribution!
P.S. please don't forget to add an Odoo version tag to the PR name next time 😄

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-852-by-ivs-cetmix-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 8044c32 into OCA:19.0 Jan 26, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at d7c6942. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants