-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[18.0][MIG] dbfilter_from_header: Migration to 18.0 #3099
[18.0][MIG] dbfilter_from_header: Migration to 18.0 #3099
Conversation
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
[MIG] migrate dbfilter_from_header to 10.0 I have also removed the ability to use HTTP_X_OPENERP_DBFILTER Signed-off-by: Carmen Bianca Bakker <[email protected]>
updates were made in OCA#1137 by @TimLai125 and small lint fixes by @yelizariev Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] Update dbfilter_from_header.pot Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] README.rst Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] Update dbfilter_from_header.pot Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] Update dbfilter_from_header.pot Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] README.rst Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] Update dbfilter_from_header.pot Signed-off-by: Carmen Bianca Bakker <[email protected]> [UPD] README.rst Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
Signed-off-by: Carmen Bianca Bakker <[email protected]>
270c090
to
dc46843
Compare
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.
All things seams ok, but test coverage.
The issue you mentionned is for OpenUpgrade script, that why this OCA module is not listed there. OpenUpgrade scripts are only for standard Odoo module. Each repository is manager by PSCs, and they can decide if this can be merge with the red ci for test coverage. PSC can be found here. |
/ocabot migration dbfilter_from_header |
Sorry @houssine78 you are not allowed to mark the addon tobe migrated. To do so you must either have push permissions on the repository, or be a declared maintainer of all modified addons. If you wish to adopt an addon and become it's maintainer, open a pull request to add your GitHub login to the |
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
@pedrobaeza this PR can be merged too. thanks ! |
/ocabot migration dbfilter_from_header |
This PR looks fantastic, let's merge it! |
Congratulations, your PR was merged at e9c7a1f. Thanks a lot for contributing to OCA. ❤️ |
No description provided.