Skip to content

[19.0][MIG] fs_attachment Migration 19.0#527

Closed
cyrilmanuel wants to merge 4 commits into
OCA:19.0from
cyrilmanuel:mig-fs_attachment-19
Closed

[19.0][MIG] fs_attachment Migration 19.0#527
cyrilmanuel wants to merge 4 commits into
OCA:19.0from
cyrilmanuel:mig-fs_attachment-19

Conversation

@cyrilmanuel

@cyrilmanuel cyrilmanuel commented Oct 16, 2025

Copy link
Copy Markdown

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

@CRogos CRogos left a comment

Copy link
Copy Markdown
Contributor

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

sbidoul commented Nov 26, 2025

Copy link
Copy Markdown
Member

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

sbidoul commented Nov 26, 2025

Copy link
Copy Markdown
Member

@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

sbidoul commented Nov 26, 2025

Copy link
Copy Markdown
Member

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.

Comment thread fs_attachment/models/fs_storage.py Outdated
@jcoux

jcoux commented Dec 1, 2025

Copy link
Copy Markdown
Contributor

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

lmignon commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

@cyrilmanuel some fixes here cyrilmanuel#3

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

lmignon commented Dec 8, 2025

Copy link
Copy Markdown
Contributor

@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