Skip to content

[14.0][IMP] sale_order_revision: 'New Revision' quick button on confirmed Sales Orders#2346

Closed
dreispt wants to merge 1 commit intoOCA:14.0from
ursais:14-dr-sale_order_revision
Closed

[14.0][IMP] sale_order_revision: 'New Revision' quick button on confirmed Sales Orders#2346
dreispt wants to merge 1 commit intoOCA:14.0from
ursais:14-dr-sale_order_revision

Conversation

@dreispt
Copy link
Copy Markdown
Member

@dreispt dreispt commented Jan 26, 2023

No description provided.

@dreispt dreispt requested a review from victoralmau January 26, 2023 21:38
@dreispt dreispt changed the title [IMP] sale_order_revision: 'New Revision' quick button on confirmed Sales Orders [14.0][IMP] sale_order_revision: 'New Revision' quick button on confirmed Sales Orders Jan 26, 2023
@rousseldenis rousseldenis added this to the 14.0 milestone Feb 17, 2023
Copy link
Copy Markdown
Contributor

@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.

LGTM. @dreispt Could you add a little test ?

@dreispt dreispt force-pushed the 14-dr-sale_order_revision branch 2 times, most recently from 365c5f3 to 1c8be71 Compare May 29, 2023 09:11
@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented May 29, 2023

@rousseldenis Test added!

@dreispt dreispt requested a review from rousseldenis May 29, 2023 09:11
@dreispt dreispt force-pushed the 14-dr-sale_order_revision branch from 1c8be71 to c93def8 Compare May 29, 2023 10:39
Comment thread sale_order_revision/models/sale_order.py
<button
name="action_cancel_create_revision"
states="sale"
string="New Revision"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I would have added something to help user to know the current one will be canceled (in string or in help at least)

Copy link
Copy Markdown
Contributor

@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.

@dreispt Could you also improve README ?

@dreispt dreispt force-pushed the 14-dr-sale_order_revision branch from d87f947 to 8d3def9 Compare December 3, 2023 15:37
@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Dec 3, 2023

README improved.

@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Dec 28, 2023

@rousseldenis Can this be merged?

@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Jan 23, 2024

Ping @rousseldenis

@dreispt
Copy link
Copy Markdown
Member Author

dreispt commented Feb 3, 2024

Based on this comment, reworked and superseded by #2930

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants