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.

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

[17.0][OU-ADD] website_links: Nothing to do #4865

Merged
merged 1 commit into from
Mar 20, 2025

Conversation

carlos-lopez-tecnativa
Copy link
Contributor

TT51556
@Tecnativa @pedrobaeza @MiquelRForgeFlow could you please review this?

I noticed that there is no migration folder for this module, even though it is present in V17. Why? However, I have done the analysis and found no changes. Is it correct to just edit docsource/modules160-170.rst, or do I need to add a migration folder for this module?

@MiquelRForgeFlow
Copy link
Contributor

Why?

When there are absolute no changes. I didn't check the code why sometimes it creates an empty analysis file and why sometimes it doesn't create the folder. I am not sure what is the difference between this two cases. I will investigate.

do I need to add a migration folder for this module?

No.

@MiquelRForgeFlow
Copy link
Contributor

/ocabot migration website_links

@MiquelRForgeFlow MiquelRForgeFlow merged commit 3034988 into OCA:17.0 Mar 20, 2025
4 checks passed
@carlos-lopez-tecnativa carlos-lopez-tecnativa deleted the 17.0-mig-website_links branch March 20, 2025 15:46
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.

2 participants