Skip to content

Porting website_cookie_notice and website_legal_page to 9.0 - #106

Closed
eLBati wants to merge 4 commits into
OCA:9.0from
eLBati:port_cookie_notice_9
Closed

eLBati wants to merge 4 commits into
OCA:9.0from
eLBati:port_cookie_notice_9

Conversation

@eLBati

@eLBati eLBati commented Nov 11, 2015

Copy link
Copy Markdown
Member

see commits

@rafaelbn rafaelbn added this to the 9.0 milestone Nov 18, 2015
@hbrunn

hbrunn commented Nov 26, 2015

Copy link
Copy Markdown
Member

👍 travis errors unrelated

@rafaelbn rafaelbn self-assigned this Dec 4, 2015
@rafaelbn

rafaelbn commented Dec 4, 2015

Copy link
Copy Markdown
Member

Rebuilind for testing

@rafaelbn

rafaelbn commented Dec 4, 2015

Copy link
Copy Markdown
Member

@eLBati runbot it's failing I cannot rebuild

@eLBati

eLBati commented Dec 10, 2015

Copy link
Copy Markdown
Member Author

@rafaelbn you should be able to test at http://3136852-106-ee2f1b.runbot2.odoo-community.org/

About travis, I am not able to reproduce the error locally (with latest revision of odoo).

@OCA/website-maintainers could you restart the build https://travis-ci.org/OCA/website/builds/90513632 ?

@rafaelbn

Copy link
Copy Markdown
Member

hi @eLBati , in you link:

10-12-2015 13-23-48

@eLBati

eLBati commented Dec 10, 2015

Copy link
Copy Markdown
Member Author

Something weird is happening to runbot

@eLBati
eLBati force-pushed the port_cookie_notice_9 branch from ee2f1bf to fb754e6 Compare December 11, 2015 14:40
@eLBati

eLBati commented Dec 11, 2015

Copy link
Copy Markdown
Member Author

Rebuilded travis, still red 💔

@pedrobaeza pedrobaeza mentioned this pull request Jan 21, 2016
34 tasks
@rafaelbn rafaelbn removed their assignment Mar 8, 2016
@antespi

antespi commented Mar 8, 2016

Copy link
Copy Markdown
Contributor

New commits have been done in branch 8.0. They could be merged here, isn't it?
https://github.com/OCA/website/commits/8.0/website_cookie_notice

@lasley

lasley commented Apr 20, 2016

Copy link
Copy Markdown
Contributor

Looks like we're missing a few commits that have been added since this PR was last worked on:

website_legal_page

website_cookie_notice

There were a few others, but they spanned multiple modules so will be hard to cherry pick in.

@eLBati

eLBati commented Apr 23, 2016

Copy link
Copy Markdown
Member Author

@antespi @lasley made another branch including last changes from 8.0 version at #195
I removed website_legal_page from the PR, as ported by #194

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.

5 participants