-
-
Notifications
You must be signed in to change notification settings - Fork 130
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
useful unfinished PRs from @flachica #40
Comments
@flachica any reason for closing all your PRs? |
I have seen that with my current workload I will not be able to improve the PR made in a reasonable time. I have also seen that there are contributions from other colleagues to whom my PR would generate many conflicts, such as #28. The #31 for example, you have to change the focus in favor of another PR. Things like that. Thanks for the interest |
OK, it's a pity to hear that your company doesn't reserve you time for OCA. |
Well, that's not entirely accurate. What I say that my current load is going to prevent me from putting hours in the short term. I prefer to close the PR and incorporate what is necessary after a while. I really liked this approach and this would make me tweak some PR's. New PR's will arrive and not only in this repository. |
Great then. |
Fix multicompany style for 13.0 #37 multicompany 13.0
Include text warnings. For example, to log computed change in 12 to 13 #31 _TEXT_WARNINGS support (+ examples)
Refactoring to convert script into classes #28 Refactoring to convert script into classes
Fix double data elements #26 properly handle
<openerp><data noupdate="0">
Replacement ir.needaction_mixin #21 r"ir.needaction_mixin": "mail.activity.mixin",
The text was updated successfully, but these errors were encountered: