Skip to content

Conversation

@Saran440
Copy link
Member

This PR similar sale_cancel_reason but this module allow free text cancel.

TODO:

  • Test Script

_has_cancel_reason = "optional" # ["no", "optional", "required"]

def _action_cancel(self):
test_condition = not config["test_enable"] or (
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not fond of these things. I'd prefer configuration on res.company to enable the behaviour.

@rousseldenis
Copy link
Contributor

This PR similar sale_cancel_reason but this module allow free text cancel.

At least, something in the README could be great in order to converge both modules in future versions. Thanks

@rousseldenis
Copy link
Contributor

/ocabot migration sale_cancel_confirm

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.

8 participants