Skip to content

[MIG] [9.0] website_menu_by_user_status - #275

Closed
hurrinico wants to merge 9 commits into
OCA:9.0from
hurrinico:9.0-mig-website_menu_by_user_status
Closed

hurrinico wants to merge 9 commits into
OCA:9.0from
hurrinico:9.0-mig-website_menu_by_user_status

Conversation

@hurrinico

Copy link
Copy Markdown

No description provided.

@pedrobaeza pedrobaeza mentioned this pull request Nov 10, 2016
34 tasks
@pedrobaeza pedrobaeza changed the title Miration website menu by user status to version 9 [MIG] [9.0] website_menu_by_user_status Nov 10, 2016
@lasley

lasley commented Nov 25, 2016

Copy link
Copy Markdown
Contributor

@hurrinico - please rebase onto the current 9.0 to fix up the SASS issues in Travis

@hurrinico
hurrinico force-pushed the 9.0-mig-website_menu_by_user_status branch from b3bffb7 to 5760f3e Compare February 17, 2017 14:04
@hurrinico

Copy link
Copy Markdown
Author

@lasley i'm quite late :D but i've rebased and the pr is green!

@eLBati eLBati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tried on runbot
http://3280408-275-5760f3.runbot2.odoo-community.org/:
removed User Not Logged from Forum menu.
Opening http://3280408-275-5760f3.runbot2.odoo-community.org/ with private session, I can see the forum menu

@tarteo

tarteo commented Mar 23, 2018

Copy link
Copy Markdown
Member

Hi, I tried to test it on runbot but unfortunately I couldn't due to an error (not related to this module):
image

@pedrobaeza

Copy link
Copy Markdown
Member

It was fixed meanwhile. You can rebase. @lasley is now out of Odoo, so don't expect him to make anything more. You can superseed this PR.

@simahawk

simahawk commented Oct 1, 2018

Copy link
Copy Markdown

@hurrinico @eLBati ping :)

@simahawk

simahawk commented Oct 1, 2018

Copy link
Copy Markdown

@hurrinico I'll provide you a base to backport via this #394
and we gonna have the same base for v8 too #509 that will be:
website_menu_permissoin to handle the real permission machinery and website_menu_by_user_status only to provide the quick flag

@eLBati

eLBati commented Oct 1, 2018

Copy link
Copy Markdown
Member

For me this can be closed

@simahawk

simahawk commented Oct 2, 2018

Copy link
Copy Markdown

@eLBati fine :)

@simahawk simahawk closed this Oct 2, 2018
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.

7 participants