Skip to content

[18.0][MIG] dbfilter_from_header: Migration to 18.0 - #3099

Merged
OCA-git-bot merged 20 commits into
OCA:18.0from
coopiteasy:18.0-mig-dbfilter_from_header
Nov 12, 2024
Merged

OCA-git-bot merged 20 commits into
OCA:18.0from
coopiteasy:18.0-mig-dbfilter_from_header

Conversation

@remytms

@remytms remytms commented Oct 29, 2024

Copy link
Copy Markdown
Contributor

No description provided.

hbrunn and others added 19 commits October 29, 2024 14:08
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
[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 <carmen@coopiteasy.be>
updates were made in OCA#1137 by @TimLai125

and small lint fixes by @yelizariev

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] Update dbfilter_from_header.pot

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] README.rst

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] Update dbfilter_from_header.pot

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] Update dbfilter_from_header.pot

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] README.rst

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] Update dbfilter_from_header.pot

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>

[UPD] README.rst

Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
Signed-off-by: Carmen Bianca Bakker <carmen@coopiteasy.be>
@remytms remytms changed the title [MIG] dbfilter_from_header: Migration to 18.0 [18.0][MIG] dbfilter_from_header: Migration to 18.0 Oct 29, 2024
@remytms
remytms force-pushed the 18.0-mig-dbfilter_from_header branch from 270c090 to dc46843 Compare October 29, 2024 14:06
@remytms
remytms marked this pull request as ready for review October 29, 2024 14:06

@mostafabarmshory mostafabarmshory left a comment

Copy link
Copy Markdown

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.

@mostafabarmshory

Copy link
Copy Markdown

@remytms

Any news?
I could not understand why this was not merged with branch 18.0.
Even not listed in #3047

@remytms

remytms commented Nov 11, 2024

Copy link
Copy Markdown
Contributor Author

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.

@houssine78

Copy link
Copy Markdown

/ocabot migration dbfilter_from_header

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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 maintainers key of its manifest.

@houssine78 houssine78 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@houssine78

Copy link
Copy Markdown

@pedrobaeza this PR can be merged too. thanks !

@pedrobaeza

Copy link
Copy Markdown
Member

/ocabot migration dbfilter_from_header
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 12, 2024
@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-3099-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request Nov 12, 2024
42 tasks
@OCA-git-bot
OCA-git-bot merged commit e7b75a1 into OCA:18.0 Nov 12, 2024
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at e9c7a1f. Thanks a lot for contributing to OCA. ❤️

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.