Skip to content

[12.0][ADD] Add a button to redirect to the webshop in the menu #549

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

Closed
wants to merge 2 commits into from

Conversation

mihien
Copy link

@mihien mihien commented May 31, 2025

@mihien mihien force-pushed the 12.0-add-beesdoo_website_shopbutton branch 5 times, most recently from 318820e to fad363b Compare June 2, 2025 17:02
@mihien mihien force-pushed the 12.0-add-beesdoo_website_shopbutton branch from fad363b to d739422 Compare June 2, 2025 17:15
@mihien mihien requested a review from huguesdk June 3, 2025 10:24
* add missing dependencies.
* use firstname and lastname fields directly.
* use existing css class and simply additional css.
* add cart icon.
* remove obsolete controller.
* add missing xml declaration.
* fix copyright statements.
@huguesdk huguesdk force-pushed the 12.0-add-beesdoo_website_shopbutton branch from 6f97098 to c9d5d99 Compare June 3, 2025 14:12
Copy link
Member

@huguesdk huguesdk left a comment

Choose a reason for hiding this comment

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

thanks for this dev, @mihien!

i’ve made some changes directly to win some time.

this is very specific to bees coop (as their webshop url is hardcoded), while this repository contains mostly generic code that can be reused. moreover, it isn’t related to the built-in odoo webshop (which is confusing). so i think that this module should go in cie-custom, and be named bees_custom_webshop_button.

@mihien mihien closed this Jun 3, 2025
@mihien
Copy link
Author

mihien commented Jun 3, 2025

Moved to here

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.

3 participants