Skip to content

Conversation

@victoralmau
Copy link
Member

@victoralmau victoralmau commented Feb 6, 2026

Avoiding dms.directory access errors in some cases

Use case:

  • Modify the user Marc Demo and define some permissions in Employees
  • Go to the employee Abigail Peterson and create her folder
  • Modify the Access group > Everyone for Employee DMS and remove the corresponding group to leave only Mitchell Admin defined as an explicit user
  • Log in with Marc Demo and try to modify some data for the employee Abigail Peterson (define a tag, for example).

Before
antes

After
despues

Please @pedrobaeza and @CarlosRoca13 can you review it?

@Tecnativa TT60793

Use case:
- Modify the user Marc Demo and define some permissions in Employees
- Go to the employee Abigail Peterson and create her folder
- Modify the Access group > Everyone for Employee DMS and remove the
corresponding group to leave only Mitchell Admin defined as an explicit user
- Log in with Marc Demo and try to modify some data for the employee
Abigail Peterson (define a tag, for example).

TT60793
@OCA-git-bot
Copy link
Contributor

Hi @CarlosRoca13,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 18.0 milestone Feb 6, 2026
@victoralmau victoralmau marked this pull request as ready for review February 9, 2026 09:26
Copy link
Contributor

@CarlosRoca13 CarlosRoca13 left a comment

Choose a reason for hiding this comment

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

Thanks!

@CarlosRoca13
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

@CarlosRoca13 The merge process could not start, because of exception 502 Server Error.

@CarlosRoca13
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-462-by-CarlosRoca13-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 6e74e3e into OCA:18.0 Feb 10, 2026
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 18.0-fix-dms_field-TT60793 branch February 10, 2026 07:13
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.

4 participants