Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][FIX] dms: Fix share action #344

Merged
merged 1 commit into from
Jun 19, 2024

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Jun 17, 2024

FWP from 15.0: #345

Allow share (directories and files) without 'Contact creation' permission

Extra changes: Replace Share buttons with actions

Steps to reproduce:

  • Go to Settings > Users & Companies > Users and create a user only with Documents > Manager permission. (User must not have the Extra Rights > Contact Creation permission).
  • Go to Documents and go into a directory form view and click on "Share" button.
  • An access error is displayed

Fixes #343

Please @pedrobaeza can you review it?

@Tecnativa

@victoralmau victoralmau changed the title [[FIX] dms: Fix share action [16.0][FIX] dms: Fix share action Jun 17, 2024
@victoralmau victoralmau marked this pull request as ready for review June 17, 2024 12:15
@pedrobaeza pedrobaeza added this to the 16.0 milestone Jun 17, 2024
@pedrobaeza
Copy link
Member

AFAIK, this works without ACL having "Contact creation" permission.

@victoralmau victoralmau marked this pull request as draft June 18, 2024 06:51
@victoralmau victoralmau marked this pull request as ready for review June 19, 2024 06:28
@victoralmau
Copy link
Member Author

Changed with v15 approach (fwp)

dms/wizards/__init__.py Outdated Show resolved Hide resolved
dms/wizards/wizard_dms_share.py Outdated Show resolved Hide resolved
…ion' permission

Extra changes: Replace Share buttons with actions

Steps to reproduce:
 - Go to Settings > Users & Companies > Users and create a user only with Documents > Manager permission.
(User must not have the Extra Rights > Contact Creation permission).
 - Go to Documents and go into a directory form view and click on "Share" button.
 - An access error is displayed
Copy link

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

👍 LGTM

@pedrobaeza
Copy link
Member

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-344-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit af11bde into OCA:16.0 Jun 19, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 16.0-fix-dms-share branch June 19, 2024 16:58
@hitrosol
Copy link

Thanks for this fix 👍

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.

5 participants