Skip to content

[ADD] sale_order_revision_stock: carry Delivery links to new revision Sales Orders#2931

Closed
dreispt wants to merge 4 commits intoOCA:14.0from
ursais:14-dr-sale_order_revision_stock
Closed

[ADD] sale_order_revision_stock: carry Delivery links to new revision Sales Orders#2931
dreispt wants to merge 4 commits intoOCA:14.0from
ursais:14-dr-sale_order_revision_stock

Conversation

@dreispt
Copy link
Copy Markdown
Member

@dreispt dreispt commented Jan 27, 2024

If the user created a default afvorite filter on the Sales Order list,
it will override the default filter set by the Revisons button.

So, the Sale Order menu Action should not be reused for this.
@dreispt dreispt marked this pull request as draft January 27, 2024 15:44
@dreispt dreispt force-pushed the 14-dr-sale_order_revision_stock branch 7 times, most recently from e4283fe to 08b74d2 Compare January 27, 2024 18:54
@dreispt dreispt marked this pull request as ready for review January 27, 2024 18:54
@dreispt dreispt requested a review from victoralmau January 27, 2024 18:55
@rousseldenis
Copy link
Copy Markdown
Contributor

@dreispt My question is : Is it normal to do revisions on sale orders that have already stock moves (and so, has already been confirmed) ?

@dreispt dreispt marked this pull request as draft February 5, 2024 15:20
@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Feb 5, 2024

@rousseldenis Good question.
The use case is caused by Portuguese Tax Authorities, that require Quotations or Sale orders sent to customers to be inalterable, signed with a cryptographic hash, and be electronically reported to them.
So if I send ou a Sales order to a customer, and later I need to make changes to it , such adding a line, or correcting something, I have to do it in a copy of the original order.

Today you need to create a copy of the SO, with all the disadvantages of doing that :-(
This is an attempt for a better solution.

@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Feb 5, 2024

PS: I could move this to l10n-portugal in case it is a bit absurd anywhere else...

@rousseldenis
Copy link
Copy Markdown
Contributor

@dreispt I would say this should be configurable (configuration parameter on company)

@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Aug 20, 2024

Superseded by #3275

@dreispt dreispt closed this Aug 20, 2024
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.

2 participants