Skip to content

Conversation

@cyrilmanuel
Copy link

@cyrilmanuel cyrilmanuel commented Oct 16, 2025

depend on #524

@cyrilmanuel cyrilmanuel changed the title [19.0][MIG] fs_storage Migration 19.0 [19.0][MIG] fs_attachment Migration 19.0 Oct 16, 2025
Copy link

@CRogos CRogos left a comment

Choose a reason for hiding this comment

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

I think you need to restart following: https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-19.0

The commit history is not preserved.

@sbidoul
Copy link
Member

sbidoul commented Nov 26, 2025

The PR looks fine (19.0 was branched off 18.0, so only the migration commits appear here), but it misses a test-requirements.txt to reference the unmerged fs_storage PR.

@sbidoul
Copy link
Member

sbidoul commented Nov 26, 2025

@cyrilmanuel can you add a test-requirements.txt like this so we can see if tests pass:

odoo-addon-server_environment @ git+https://github.com/OCA/server-env@refs/pull/247/head#subdirectory=server_environment
odoo-addon-fs_storage @ git+https://github.com/OCA/storage@refs/pull/524/head#subdirectory=fs_storage

@sbidoul
Copy link
Member

sbidoul commented Nov 26, 2025

There are also a few pre-commit issues that need addressing. The empty domains errors in pylint can be ignored with a # pylint: disable=... comment.

@jcoux
Copy link
Contributor

jcoux commented Dec 1, 2025

Did few fixes on cyrilmanuel#2
Not sure all is fixed yet.
Should be integrated here by @cyrilmanuel in next days.

fixup! fixup! [19.0][MIG] fs_attachment Migration 19.0
@lmignon
Copy link
Contributor

lmignon commented Dec 8, 2025

@cyrilmanuel some fixes here cyrilmanuel#3

@lmignon lmignon mentioned this pull request Dec 8, 2025
2 tasks
@lmignon
Copy link
Contributor

lmignon commented Dec 8, 2025

@cyrilmanuel I moved forward with your great work in #544 ....

@cyrilmanuel cyrilmanuel closed this Dec 9, 2025
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.

5 participants