Skip to content

9.0 fix website logo - #295

Merged
pedrobaeza merged 2 commits into
OCA:9.0from
StephanRozendaal:9.0-fix-website_logo
Dec 28, 2016
Merged

pedrobaeza merged 2 commits into
OCA:9.0from
StephanRozendaal:9.0-fix-website_logo

Conversation

@StephanRozendaal

Copy link
Copy Markdown

website_logo module was not working, and the read me not up to date, as reported in: #289

Use domain field to match a logo by website instead of the name field of the website
model.
The logo is now configured from 'Website admin' instead of 'Company settings'.
@pedrobaeza

Copy link
Copy Markdown
Member

Travis error unrelated

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

LGTM

@pedrobaeza
pedrobaeza merged commit 2c46824 into OCA:9.0 Dec 28, 2016
chienandalu pushed a commit to Tecnativa/website that referenced this pull request Jun 23, 2017
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
chienandalu pushed a commit to Tecnativa/website that referenced this pull request Jun 23, 2017
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
lasley pushed a commit to LasLabs/website that referenced this pull request Aug 11, 2017
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
ernestotejeda pushed a commit to ernestotejeda/website that referenced this pull request Jun 4, 2018
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
ernestotejeda pushed a commit to ernestotejeda/website that referenced this pull request Jun 13, 2018
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
HviorForgeFlow pushed a commit to ForgeFlow/website that referenced this pull request Oct 16, 2018
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
ernestotejeda pushed a commit to Tecnativa/website that referenced this pull request May 9, 2019
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
OCA-git-bot pushed a commit that referenced this pull request Jun 27, 2019
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
pedrobaeza pushed a commit to Tecnativa/website that referenced this pull request Jun 27, 2019
* [FIX] website_logo:  show logo by domain

Use domain field to match a logo by website instead of the name field of the website
model.

* [FIX] website_logo update readme

The logo is now configured from 'Website admin' instead of 'Company settings'.
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.

3 participants