Skip to content

[19.0][MIG] agreement_legal: Migration to 19.0 - #119

Closed
max3903 wants to merge 2 commits into
OCA:19.0from
ursais:19.0-mig-agreement_legal
Closed

[19.0][MIG] agreement_legal: Migration to 19.0#119
max3903 wants to merge 2 commits into
OCA:19.0from
ursais:19.0-mig-agreement_legal

Conversation

@max3903

@max3903 max3903 commented Jul 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Migrate agreement_legal from 18.0 to 19.0
  • Update security groups to res.groups.privilege / group_ids pattern
  • Apply 19.0 framework cleanups (settings action, translations API, JS assets)
  • Expand tests for 100% Python coverage

Dependencies

  • agreement — merged on 19.0

Test plan

  • pre-commit run -a
  • Module tests in OCA CI container (44 tests, 0 failures)
  • Python coverage 100% on agreement_legal

Made with Cursor

@max3903

max3903 commented Jul 3, 2026

Copy link
Copy Markdown
Member Author

Dependencies

  • agreement — available on 19.0 (no blocking dependency)

/ocabot migration agreement_legal

@max3903 max3903 added this to the 19.0 milestone Jul 3, 2026
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 3, 2026
6 tasks
max3903 and others added 2 commits July 3, 2026 09:24
Migrate from 18.0 with res.groups.privilege security, 19.0 API
cleanups, expanded tests (100% coverage), and per-model demo data.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@max3903
max3903 force-pushed the 19.0-mig-agreement_legal branch from d111380 to 672ef38 Compare July 3, 2026 15:24
@max3903

max3903 commented Jul 3, 2026

Copy link
Copy Markdown
Member Author

@CRogos @SyncMasta

@CRogos CRogos left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Migration history missing.
I would also like to extract the product part into a agreement_legal_product module.
We have already something prepared. If you wait till next week, we create a migration of this module.

@CRogos

CRogos commented Jul 3, 2026

Copy link
Copy Markdown

Can we switch to this PR: #120

@max3903 max3903 closed this Jul 5, 2026
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.

3 participants