Skip to content

[IMP] bike_shop: improve design of demo website - #2753

Open
eswa-odoo wants to merge 1 commit into
odoo:saas-19.4from
odoo-dev:saas-19.4-bike_shop-website-eswa
Open

[IMP] bike_shop: improve design of demo website#2753
eswa-odoo wants to merge 1 commit into
odoo:saas-19.4from
odoo-dev:saas-19.4-bike_shop-website-eswa

Conversation

@eswa-odoo

Copy link
Copy Markdown

This commit adds a demo website into the /bike_shop industry module. When the user installs the industry with demo data, it creates a second website with a cutting edge look and feel to demonstrate what users can achieve with the Website Builder.

task-5477795

@eswa-odoo
eswa-odoo requested a review from bram-odoo August 10, 2026 12:54
@eswa-odoo eswa-odoo self-assigned this Aug 10, 2026
@robodoo

robodoo commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Pull request status dashboard

@bram-odoo bram-odoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey @eswa-odoo, here is my review :-) Congratulations for this top-notch gems!

Comment thread bike_shop/demo/website/pages/about.xml Outdated
Comment thread bike_shop/demo/website/pages/about.xml Outdated
Comment thread bike_shop/demo/website/pages/about.xml Outdated
Comment thread bike_shop/demo/website/pages/about.xml Outdated
Comment thread bike_shop/demo/website/pages/about.xml Outdated
Comment thread bike_shop/demo/website/views/website_templates.xml Outdated
Comment thread bike_shop/demo/website/server_actions_dynamic_filters.xml Outdated
Comment thread bike_shop/demo/website/website.xml Outdated
Comment thread bike_shop/demo/website/website.xml Outdated
Comment thread industry.sh Outdated
@bram-odoo

Copy link
Copy Markdown
Contributor

I forgot something:
Screenshot 2026-08-10 at 16 05 32

Is it normal that the ressource is shown (as it can't be modified by the end-user) or is it just because I'm logged in as an administrator?

@bram-odoo bram-odoo closed this Aug 10, 2026
@bram-odoo bram-odoo reopened this Aug 10, 2026
@eswa-odoo
eswa-odoo force-pushed the saas-19.4-bike_shop-website-eswa branch 2 times, most recently from 469d2a1 to f9c8f0b Compare August 20, 2026 06:27
Comment thread bike_shop/demo/website/website.xml Outdated
Comment thread bike_shop/static/description/favicon.png
@tocodoo
tocodoo force-pushed the saas-19.4-bike_shop-website-eswa branch from 4927e45 to 229cf81 Compare August 20, 2026 11:24
This commit adds a demo website into the `/bike_shop` industry module. When the user installs the industry with demo data, it creates a second website with a cutting edge look and feel to demonstrate what users can achieve with the Website Builder.

task-5477795
@tocodoo
tocodoo force-pushed the saas-19.4-bike_shop-website-eswa branch from 229cf81 to 9409ba9 Compare August 20, 2026 11:25
@bram-odoo
bram-odoo marked this pull request as ready for review August 21, 2026 14:00
@C3POdoo
C3POdoo requested a review from a team August 21, 2026 14:02

@vava-odoo vava-odoo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not much to say, nicely done
have a look at the warning here https://runbot.odoo.com/runbot/build/123243883

  • following comments

<!-- `data/qweb_views.xml` binds this form to `base.default_website`, which still
points to Odoo's own website when the data files are loaded. Rebind it to the
demo website created above. -->
<record id="team_form_2" model="ir.ui.view">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

shouldn't it be duplicated instead?

Comment thread bike_shop/demo/website/pages/home.xml
Comment thread bike_shop/demo/website/views/website_templates.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants