Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][ADD] sale_rental #45

Closed
wants to merge 2 commits into from
Closed

Conversation

coffeinisfun
Copy link

Add the module sale_rental to vertical-rental. This module was in sale-workflow in previous versions, but it was suggested that it is moved to vertical-rental starting with version 16.0
see this PR: OCA/sale-workflow#2325
the actual migration work was done by @Murtaza-OSI

@coffeinisfun coffeinisfun changed the title [ADD] sale_rental [16.0][ADD] sale_rental Feb 28, 2024
@pedrobaeza
Copy link
Member

Please preserve commit history. You should change the command including both remotes.

And check also CI.

@coffeinisfun
Copy link
Author

Please preserve commit history. You should change the command including both remotes.

And check also CI.

Ok, that seems like quite the effort since we are moving a module between repos here. The original PR has 61 commits and not all of can be merged into the repo. I have to take a look at all the commits and cherrypick accordingly. I can try this, except if you have a better idea

@coffeinisfun
Copy link
Author

New PR with preserved commit history: #46

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