Skip to content

[15.0][MIG] base_group_backend: Migration to 15.0#359

Open
AungKoKoLin1997 wants to merge 17 commits intoOCA:15.0from
qrtl:15.0-mig-base_group_backend
Open

[15.0][MIG] base_group_backend: Migration to 15.0#359
AungKoKoLin1997 wants to merge 17 commits intoOCA:15.0from
qrtl:15.0-mig-base_group_backend

Conversation

@AungKoKoLin1997
Copy link
Copy Markdown

@qrtl QT4975

Copy link
Copy Markdown

@kanda999 kanda999 left a comment

Choose a reason for hiding this comment

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

Functioanl test: It works as expected in my env

Copy link
Copy Markdown
Contributor

@BhaveshHeliconia BhaveshHeliconia left a comment

Choose a reason for hiding this comment

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

Functional review LGTM!

@github-actions
Copy link
Copy Markdown

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.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 28, 2025
petrus-v and others added 17 commits January 6, 2026 08:31
…up_backend

As a developer we have to keep in mind using this module and grant a user with 's group is
equivalent to grant 's group everywhere  has been used.

Co-authored-by: Jean-Charles Drubay <jcdrubay@users.noreply.github.com>
FIX applies pre-commit
IMPL adds test for share of backend user
FIX pre-commit pass
IMPL removes mail.activity.mixin from dummy model because it is not needed for the test
IMPL renames and divide the base_group_backend into 2 groups one that provide the basic rights and another that allow login in the app
IMPL changes backend ui users to a user type
FIX pre-commit pass
FIX removes useless imports
FIX adds share to group_backend_ui_users
IMPL adds mail_channel to access rights
FIX tests now working
FIX pre-commit pass
Currently translated at 100.0% (20 of 20 strings)

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

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

Translation: server-backend-16.0/server-backend-16.0-base_group_backend
Translate-URL: https://translation.odoo-community.org/projects/server-backend-16-0/server-backend-16-0-base_group_backend/it/
@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 15.0-mig-base_group_backend branch from 893c068 to 4d8693f Compare January 6, 2026 08:31
@github-actions github-actions Bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jan 11, 2026
@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@dreispt
Copy link
Copy Markdown
Member

dreispt commented Apr 18, 2026

/ocabot migration base_group_backend
/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone Apr 18, 2026
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 15.0-ocabot-merge-pr-359-by-dreispt-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Apr 18, 2026
Signed-off-by dreispt
@OCA-git-bot OCA-git-bot mentioned this pull request Apr 18, 2026
10 tasks
@OCA-git-bot
Copy link
Copy Markdown
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-359-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@OCA-git-bot
Copy link
Copy Markdown
Contributor

@dreispt your merge command was aborted due to failed check(s), which you can inspect on this commit of 15.0-ocabot-merge-pr-359-by-dreispt-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

@AungKoKoLin1997
Copy link
Copy Markdown
Author

@dreispt Seems like the merging was failed because of pre-commit issue.
I create a PR for copier update. Please check it.
#451

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.

10 participants