Skip to content

[19.0][FIX] project_task_code: allow portal users to create tasks - #1875

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
ForgeFlow:19.0-fix-project_task_code
Nov 7, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:19.0from
ForgeFlow:19.0-fix-project_task_code

Conversation

@ThiagoMForgeFlow

Copy link
Copy Markdown
Contributor

In 18.0 the code field was portal writable (SELF_WRITABLE_FIELDS), but it was dropped during the 19.0 migration.
Since create always sets code from the sequence, task creation by a portal user raises AccessError on field code.

This restores it through TASK_PORTAL_WRITABLE_FIELDS and adds a portal test.

@ForgeFlow

The code field was dropped from the portal writable fields during
the 19.0 migration, so task creation from the portal raised AccessError.

@LoisRForgeFlow LoisRForgeFlow 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.

LGTM 👍

@NICO-SOLUTIONS NICO-SOLUTIONS left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Code review. LGTM

@AaronHForgeFlow AaronHForgeFlow 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.

👍

@leemannd

leemannd commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-1875-by-leemannd-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit bd06724 into OCA:19.0 Nov 7, 2026
7 of 8 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at e2d8d6d. Thanks a lot for contributing to OCA. ❤️

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.

6 participants