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][MIG] shopinvader_restapi: Migration to 16.0 #1403

Merged

Conversation

qgroulard
Copy link
Contributor

@qgroulard qgroulard commented Sep 7, 2023

Based on #1382

string="Available partner titles",
default=lambda self: self._default_partner_title_ids(),
)
partner_industry_ids = fields.Many2many(
Copy link
Contributor

Choose a reason for hiding this comment

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

can we move this one to a dedicated module ?

@hparfr
Copy link
Contributor

hparfr commented Sep 11, 2023

there is no shopinvader_backend.xml

~~(note: I'm not conviced shopinvader_backend should be in this module, but it should be somewhere :) ) ~~
it will be probably here in the end

@qgroulard qgroulard force-pushed the 16.0-mig-shopinvader_restapi-qgr branch 3 times, most recently from 137704f to 2c08306 Compare September 19, 2023 10:58
@sebastienbeau sebastienbeau added this to the 16.0 milestone Sep 25, 2023
@lmignon lmignon force-pushed the 16.0-mig-shopinvader_restapi-qgr branch from 87be8b0 to 1aaa95c Compare October 13, 2023 13:12
The status will be changed as soon as a first implementation will be deployed in production
@lmignon
Copy link
Collaborator

lmignon commented Oct 13, 2023

/ocabot merge nobump

@shopinvader-git-bot
Copy link

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-1403-by-lmignon-bump-nobump, awaiting test results.

@shopinvader-git-bot shopinvader-git-bot merged commit b0ab41a into shopinvader:16.0 Oct 13, 2023
3 checks passed
@shopinvader-git-bot
Copy link

Congratulations, your PR was merged at 10a15b4. Thanks a lot for contributing to shopinvader. ❤️

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.

None yet

7 participants