Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[16.0][OU-ADD] website_profile: Migration to 16.0 #4177

Merged
merged 1 commit into from
Mar 7, 2024
Merged

[16.0][OU-ADD] website_profile: Migration to 16.0 #4177

merged 1 commit into from
Mar 7, 2024

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Oct 18, 2023 •
edited
Loading

Migration to 16.0

Depends on:

@Tecnativa TT45247

@legalsylvain
Copy link
Contributor

legalsylvain commented Oct 18, 2023 •
edited
Loading

/ocabot migration website_profile

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Oct 18, 2023
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 18, 2023
Open
chienandalu
chienandalu reviewed Oct 19, 2023
View reviewed changes
Copy link
Member

@chienandalu chienandalu left a comment •
edited by pedrobaeza
Loading

Choose a reason for hiding this comment

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

Depends on

@legalsylvain legalsylvain mentioned this pull request Oct 20, 2023
Merged
@legalsylvain legalsylvain mentioned this pull request Oct 20, 2023
Merged
@pedrobaeza pedrobaeza force-pushed the 16.0-mig-website_profile branch 2 times, most recently from 341b845 to cf1ff0e Compare March 7, 2024 20:21
@pedrobaeza
Copy link
Member

pedrobaeza commented Mar 7, 2024 •
edited
Loading

I have pushed a proper version. @victoralmau, some comments for future scripts:

  • noupdate=0 are removed by ORM. You don't have to explicitly remove in the scripts.
  • The syntax for the delete translations method is to put the XML-ID, and optionally the fields, but not only the fields.
  • The path of the file was not correct.

@victoralmau @pedrobaeza
[OU-ADD] website_profile: Migration to 16.0
5d96c73 
TT45247

Co-Authored-By: Pedro M. Baeza <[email protected]>
@pedrobaeza pedrobaeza merged commit 7d9d3b2 into OCA:16.0 Mar 7, 2024
2 checks passed
@pedrobaeza pedrobaeza deleted the 16.0-mig-website_profile branch March 7, 2024 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chienandalu chienandalu chienandalu left review comments

Assignees
No one assigned
Labels
Dependency OK
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.
5 participants
@victoralmau @legalsylvain @pedrobaeza @chienandalu @OCA-git-bot