Skip to content

[15.0][OU-ADD] mass_mailing_event_track: Nothing to do#4044

Merged
pedrobaeza merged 2 commits intoOCA:15.0from
Tecnativa:15.0-ou_add-mass_mailing_event_track
Jul 5, 2023
Merged

[15.0][OU-ADD] mass_mailing_event_track: Nothing to do#4044
pedrobaeza merged 2 commits intoOCA:15.0from
Tecnativa:15.0-ou_add-mass_mailing_event_track

Conversation

@pedrobaeza
Copy link
Member

Even no analysis file.

@Tecnativa TT41016

@pedrobaeza
Copy link
Member Author

/ocabot migration mass_mailing_event_track

@pedrobaeza
Copy link
Member Author

It's not blocked by dependency, @legalsylvain. It's simply a bad interaction, as we do a trick in the CI installing all modules in the previous version (and thus, having the records), and then marking them as uninstalled by SQL. I'm going the straight way not doing the conflicting XML-IDs renaming. It was just completeness effort.

@pedrobaeza pedrobaeza force-pushed the 15.0-ou_add-mass_mailing_event_track branch from 525f0ea to fac44d2 Compare July 5, 2023 09:25
@pedrobaeza pedrobaeza merged commit fcbfb21 into OCA:15.0 Jul 5, 2023
@pedrobaeza pedrobaeza deleted the 15.0-ou_add-mass_mailing_event_track branch July 5, 2023 09:36
@legalsylvain
Copy link
Contributor

legalsylvain commented Jul 5, 2023

I didn't put "Blocked by Dependency". the script I ran did it !
(#3979, feedback welcome BTW)

It is based on the parsing of the file https://github.com/OCA/OpenUpgrade/blob/15.0/docsource/modules140-150.rst. For instance, website_event_track is marked as TODO.

After investigation, it is because you forgot to update the coverage file in this PR. #3751

This trivial PR should fix the problem : #4049

@pedrobaeza
Copy link
Member Author

OK, thank you for checking.

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