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

Make web_responsive work with Odoo Enterprise #2650

Open
gpothier opened this issue Oct 30, 2023 · 3 comments
Open

Make web_responsive work with Odoo Enterprise #2650

gpothier opened this issue Oct 30, 2023 · 3 comments

Comments

@gpothier
Copy link

gpothier commented Oct 30, 2023

[EDIT]: split is actually not necessary, see comment below.

Is your feature request related to a problem?
Yes: it is currently not possible to install the web_responsive module if web_enterprise is also installed. As far as I understand, both modules would only conflict with respect to the full-screen Apps menu; the web_responsive module has many other enhancements that should not conflict with web_enterprise, and that would be very welcome on an Enterprise install.

Describe the solution you'd like
Split the module into one that is compatible with web_enterprise, and another one that is not

Describe alternatives you've considered
I will have a go at doing the splitting and submit a merge request, if it is of interest to the community.

@gpothier gpothier changed the title Split web_responsive into two modules so that most features work with Odoo Enterprise Make web_responsive work with Odoo Enterprise Oct 30, 2023
@gpothier
Copy link
Author

Following up on this: the web_responsive module as it is actually works fine alongside web_enterprise (only removing the exclusion in the manifest). So I changed the title of the issue, as I think it is not needed to split the module into two. If requested I could create a merge request for just the change in the manifest.

What would be extra nice, though, is to have this module's Apps menu (with the search feature) instead of the one of web_enterprise. I will have a look at how to do this.

@legalsylvain
Copy link
Contributor

hi @gpothier Thanks for your investigation !
Once all analyzed, PR is welcome to improve OCA module.
regards.

@gpothier
Copy link
Author

gpothier commented Nov 1, 2023

PR #2652 created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants