-
-
Notifications
You must be signed in to change notification settings - Fork 698
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[16.0][OU-ADD] website #3949
[16.0][OU-ADD] website #3949
Conversation
0c53f5d
to
da6e970
Compare
/ocabot migration website |
ace3dbb
to
3372993
Compare
On this version, Odoo has upgraded Bootstrap from 4 to 5, so similar changes than in v12 would be needed:
|
3372993
to
740d2a7
Compare
016a4b0
to
c0334d6
Compare
c0334d6
to
63f5ca9
Compare
f6bd7d4
to
d24555d
Compare
d24555d
to
e428dc8
Compare
) | ||
|
||
|
||
def keep_the_first_domain_when_duplicate(env): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why need this, do not understand ?
e428dc8
to
c8b9dcb
Compare
592515f
to
c53b8b1
Compare
We are working on a separate repository, and once the code is completed, we will create a pull request to your repository. |
c87d236
to
81ca47e
Compare
Ran an upgrade from 15.0 with website. Generally ok but came across a strange issue when trying to edit header or footer on WYSIWYG editor afterwards. Not sure if it was OpenUpgrade or an Odoo bug so reported it as an issue on there. |
@chrisandrewmann I suggest you regenerate asset after migrating, if not working then i'll try to reproduce it at my local |
Thanks @duong77476 for the quick reply. I've tried regenerating asset bundles a couple of times. Update 30/09/23: I also tested an upgrade to v16 using the same DB but via the official Odoo upgrade platform, this issue does not appear so must be something OpenUpgrade specific regards website |
oki thanks, i'll check it out later |
In testing a v13-v16 migration I received an error in the 15-16 step. A prior xml id with the name Restricted Editor already existed with xml id |
This has happened to two different databases after migration to 14.0 > 15.0 >16.0. It seems to be caused by changes in 'website.layout' view. I've not yet pinpointed the exact change causing it as there are many, easier to just replace the arch completely with 16 version. |
Thank you everyone, i will resolve conflict today |
05111ec
to
36fe483
Compare
Done |
36fe483
to
e87daf3
Compare
I have pushed a rebased version improving a bit the documentation on the work file, and making a backup of the arch_db column before updating it, as OpenUpgrade should try to be as conservative as possible. @chienandalu we will need later to add the same HTML fields conversion (BS4 > BS5 this time) that we have made in previous version, but there's still the problem with the malformed HTMLs. |
No description provided.