-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
[ADD] fleet_vehicle_purchase #128
Conversation
8ee0d67
to
ea4c067
Compare
ea4c067
to
7219e94
Compare
runboat functional test fails, trying to associate a veichle to a purchase order Error: Traceback (most recent call last): The above exception was the direct cause of the following exception: Traceback (most recent call last): |
I see the issue, but it is unrelated to this module. Obviously, If you go to services and try to change the vehicle, the same error is raised. Anyway, I will check why this happens |
the patch solved the error, could u rebase? and, what do u think about add vehicle field to the purchase PDF reports? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks! :)
@etobella Can you do a rebase, please? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. ❤️
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at c2ce6ac. Thanks a lot for contributing to OCA. ❤️ |
Allow to define fleet vehicles on Purchase Orders in order to inherit them properly