[MIG] base_dav: Migration to 17.0 by jaydeep32 · Pull Request #269 · OCA/server-backend

jaydeep32 · 2024-02-14T16:33:52Z

To install and use this module, use the right version of radicale==2.1.12 python package.
This PR for this #254

Currently translated at 100.0% (62 of 62 strings) Translation: server-backend-17.0/server-backend-17.0-base_user_role Translate-URL: https://translation.odoo-community.org/projects/server-backend-17-0/server-backend-17-0-base_user_role/es/

Currently translated at 100.0% (62 of 62 strings) Translation: server-backend-17.0/server-backend-17.0-base_user_role Translate-URL: https://translation.odoo-community.org/projects/server-backend-17-0/server-backend-17-0-base_user_role/it/

Currently translated at 100.0% (11 of 11 strings) Translation: server-backend-14.0/server-backend-14.0-base_user_role_company Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_user_role_company/it/

Issue found on logout / relogin. The user groups were applied correctly, but the main menu showed apps the user did not have access to. This was related to the menu caching mechanisn, that was disabled here.

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-backend-14.0/server-backend-14.0-base_user_role_company Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_user_role_company/

Currently translated at 90.9% (10 of 11 strings) Translation: server-backend-14.0/server-backend-14.0-base_user_role_company Translate-URL: https://translation.odoo-community.org/projects/server-backend-14-0/server-backend-14-0-base_user_role_company/it/

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: server-backend-16.0/server-backend-16.0-base_user_role_company Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_user_role_company/

Currently translated at 100.0% (8 of 8 strings) Translation: server-backend-16.0/server-backend-16.0-base_user_role_company Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_user_role_company/es/

Co-authored-by: Yannick Vaucher <yannick.vaucher@camptocamp.com>

…rver-backend into 17.0-mig-base_dav

jaydeep32 · 2024-03-20T14:23:16Z

Hello @dreispt Now is it ok? I have added those commits from 11.0 to this one.

dreispt · 2024-03-20T14:41:04Z

    ],
    "external_dependencies": {
-        "python": ["radicale"],
+        "python": ["radicale==2.1.12"],


Can this be >= ?

Here, I use this specific module version because this module's code is only compatible with this 2.1.12 or older version of radical. If you check this #254 issue here, @AquaMCU has mentioned that he is having this strange error in the log,
so I solved that by changing the module version to radicale==2.1.12.

Also, if possible, can we add @AquaMCU to the discussion since he is the one who has migrated the module?
Actually, he was not able to create PR, so I simply made that for him

dreispt · 2024-03-20T14:43:12Z

        if not self:
            return self.env["unknown"]
        self.ensure_one()
+        self = self.with_user(SUPERUSER_ID)


This raises security concerns, why is this required?

amh-mw

@jaydeep32 This doesn't look right. Why are there commits specific to the base_user_role_company module? I second @dreispt 's comment that you need to start fresh with a clean local folder and follow the steps in the migration documentation.

dreispt · 2024-03-20T14:45:42Z

@jaydeep32 Great work, you made it!
Now I can see your migration changes.
I made a couple of questions, thanks.

dreispt · 2024-03-20T14:58:41Z

@amh-mw You're right.
@jaydeep32 I ran the migration process myself and applied your changes, here #276
This has the correct history.

I suggest you fetch my branch, git reset this branch to it, and continue from there.

flotho

thanks for this PR.
I thinks some ACL are missing for http://oca-server-backend-17-0-pr269-360893ee9167.runboat.odoo-community.org/web?debug=1#id=1&cids=1&menu_id=65&action=77&model=dav.collection&view_type=form

flotho · 2024-07-11T19:27:26Z

thanks for this PR. I thinks some ACL are missing for http://oca-server-backend-17-0-pr269-360893ee9167.runboat.odoo-community.org/web?debug=1#id=1&cids=1&menu_id=65&action=77&model=dav.collection&view_type=form

mostly impossible to configure this in the UI

eLBati · 2024-07-29T10:44:27Z

Should we move to #276 ?

josselex · 2024-10-05T08:55:10Z

Is there any progress on it? Maybe even upgrading it to Odoo 18? Can we sponsor this?

Syncing from upstream OCA/server-backend (12.0)

github-actions · 2025-02-02T12:17:47Z

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

Katulos · 2025-12-12T20:22:10Z

Is there any progress on it? Maybe even upgrading it to Odoo 18? Can we sponsor this?

Hello.
Porting to 18.0 is no problem. But as we see, OCA isn't accepting pull requests.

Aldeigja · 2026-03-06T17:32:13Z

Is there any progress on it? Maybe even upgrading it to Odoo 18? Can we sponsor this?

@josselex Please take a look at #431 functional testing and code review would be very helpful.

jaydeep32 and others added 30 commits February 14, 2024 11:31

[MIG] base_dav: Migration to 17.0

7844e6e

[IMP] base_dav: Some test cases fixes,odoo tools safe_eval fixes.

5e3179e

[IMP] base_dav: pre-commit auto fixes

a244bd5

[UPD] base_dav: python package version

10d533d

[UPD] base_dav: Module external_dependencies with version.

804cce3

Translated using Weblate (Spanish)

a14edea

Currently translated at 100.0% (62 of 62 strings) Translation: server-backend-17.0/server-backend-17.0-base_user_role Translate-URL: https://translation.odoo-community.org/projects/server-backend-17-0/server-backend-17-0-base_user_role/es/

Translated using Weblate (Italian)

2efc7ea

Currently translated at 100.0% (62 of 62 strings) Translation: server-backend-17.0/server-backend-17.0-base_user_role Translate-URL: https://translation.odoo-community.org/projects/server-backend-17-0/server-backend-17-0-base_user_role/it/

[ADD] base_user_role_company: add per company user roles

a6546d7

[UPD] Update base_user_role_company.pot

e063507

[UPD] README.rst

0fba526

[ADD] icon.png

5a15310

[FIX] base_user_role_company: wrong xpath

2976844

base_user_role_company 14.0.1.1.0

9752e9c

Added translation using Weblate (Italian)

b7c3aa6

[FIX] base_user_role_company: roles not properly applied on login

31b92c7

[FIX] base_user_role_company: wrong menus on re-login

b0b99e0

Issue found on logout / relogin. The user groups were applied correctly, but the main menu showed apps the user did not have access to. This was related to the menu caching mechanisn, that was disabled here.

[FIX] base_user_role_company: fix tests

e81489f

[UPD] Update base_user_role_company.pot

8e0628d

base_user_role_company 14.0.2.0.0

677b853

base_user_role_company 14.0.2.0.1

20abc76

[IMP] base_user_role_company: pre-commit stuff

581fb46

[MIG] base_user_role_company: Migration to 16.0

81827d8

[UPD] Update base_user_role_company.pot

a06728a

[UPD] README.rst

4f3f215

Added translation using Weblate (Spanish)

2d62260

fkantelberg and others added 12 commits March 20, 2024 10:06

Fix CI errors

54b7a7f

Mute radicale logger for testing

dd23c30

Use the collection as first argument in the creation of the items

e194c4a

Use related model_id field to build domain

f1a711c

Set default id for fields

f9b5973

Co-authored-by: Yannick Vaucher <yannick.vaucher@camptocamp.com>

Add missing quotation

555cf49

[UPD] Update base_dav.pot

b5b9e20

[UPD] README.rst

8b51c3e

[UPD] README.rst

3911e59

[IMP] base_dav: pre-commit auto fixes

596854d

[MIG] base_dav: Migration to 17.0

9106fc5

Merge branch '17.0-mig-base_dav' of https://github.com/archeti-org/se…

360893e

…rver-backend into 17.0-mig-base_dav

dreispt reviewed Mar 20, 2024

View reviewed changes

amh-mw suggested changes Mar 20, 2024

View reviewed changes

dreispt linked an issue Mar 20, 2024 that may be closed by this pull request

base_dav dead? #254

Closed

dreispt mentioned this pull request Mar 20, 2024

[MIG] base_dav: Migration to 17.0 #276

Draft

flotho suggested changes Jul 11, 2024

View reviewed changes

SiesslPhillip pushed a commit to grueneerde/OCA-server-backend that referenced this pull request Nov 20, 2024

Merge pull request OCA#269 from OCA/12.0

d449ddb

Syncing from upstream OCA/server-backend (12.0)

github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 2, 2025

github-actions Bot closed this Mar 9, 2025

Uh oh!

Conversation

jaydeep32 commented Feb 14, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jaydeep32 commented Mar 20, 2024

Uh oh!

dreispt Mar 20, 2024

Choose a reason for hiding this comment

Uh oh!

jaydeep32 Mar 20, 2024

Choose a reason for hiding this comment

Uh oh!

dreispt Mar 20, 2024

Choose a reason for hiding this comment

Uh oh!

amh-mw left a comment

Choose a reason for hiding this comment

Uh oh!

dreispt commented Mar 20, 2024

Uh oh!

dreispt commented Mar 20, 2024

Uh oh!

flotho left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

flotho commented Jul 11, 2024

Uh oh!

eLBati commented Jul 29, 2024

Uh oh!

josselex commented Oct 5, 2024

Uh oh!

github-actions Bot commented Feb 2, 2025

Uh oh!

Katulos commented Dec 12, 2025

Uh oh!

Aldeigja commented Mar 6, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

jaydeep32 commented Feb 14, 2024 •

edited

Loading

flotho left a comment •

edited

Loading