Skip to content

[FIX] website_multi_theme: Proper checking of special views inheritance - #433

Merged
pedrobaeza merged 1 commit into
OCA:10.0from
Tecnativa:10.0-website_multi_theme-false_positives
Mar 9, 2018
Merged

pedrobaeza merged 1 commit into
OCA:10.0from
Tecnativa:10.0-website_multi_theme-false_positives

Conversation

@yajo

@yajo yajo commented Mar 9, 2018

Copy link
Copy Markdown
Member

While reviewing #430 I saw this bug, so here's the quick fix.

Some of these blocks were never hit, so here I'm fixing the false negatives by using recordset operations.

@Tecnativa

@yajo yajo self-assigned this Mar 9, 2018
@yajo yajo added this to the 10.0 milestone Mar 9, 2018

@pedrobaeza pedrobaeza 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.

Fast-tracking this

@pedrobaeza
pedrobaeza merged commit ad84e38 into OCA:10.0 Mar 9, 2018
@pedrobaeza
pedrobaeza deleted the 10.0-website_multi_theme-false_positives branch March 9, 2018 09:55
@yelizariev

Copy link
Copy Markdown
Member

@yajo Shall it be applied for 11.0?

@yelizariev

Copy link
Copy Markdown
Member

#435

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