[WIP] Adding filter menu to differentiate between Odoo CMS and OCA we…

[PCatinean]

Work in progress to filter different menus for both CMS's

[PCatinean]

@simahawk I think this PR is ready. Let me know if anything needs changing or fixing.

[leemannd]

<odoo> without <data>

[leemannd]

Only a small change but seems good to me 👍

[simahawk]

@PCatinean can you fix the small conflict in menu.xml. Should be just a matter of replacing <openerp /> with <odoo />, thanks.

[simahawk]

@PCatinean any way to make this readable? :)

Maybe we can move this logic to a website helper method?

[PCatinean]

I copied and pasted the original navbar template from stock Odoo. I thought about using the original template but it's highly unlikely it will work.

Methods in the website helper might be a bit overkill, I set some variables at the start.

What do you think?

[simahawk]

totally fine :)

On Tue, Oct 11, 2016 at 9:18 PM, Paul Catinean [email protected]
wrote:

@PCatinean commented on this pull request.

In website_cms/templates/menu.xml
#1:

@@ -42,7 +41,28 @@

-
-
+

I copied and pasted the original navbar template from stock Odoo. I
thought about using the original template but it's highly unlikely it will
work.

Methods in the website helper might be a bit overkill, I set some
variables at the start.

What do you think?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVMDR89rw-nOTV73gLWhELriBypyfJXks5qy-DrgaJpZM4KMxe8
.