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.

Sign up for GitHub

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

Jump to bottom

[16.0][IMP] dms: Add ACLs to dms.storage for users with the basic permission + Add human_size field to directories and files #303

Merged
merged 2 commits into from
Feb 27, 2024
Merged

[16.0][IMP] dms: Add ACLs to dms.storage for users with the basic permission + Add human_size field to directories and files #303

merged 2 commits into from
Feb 27, 2024

Conversation

victoralmau
Copy link
Member

@victoralmau victoralmau commented Feb 19, 2024 •
edited
Loading

FWP from 15.0: #279

Changes done:

  • Add ACLs to dms.storage for users with the basic permission.
  • Add human_size field to directories and files

Please @pedrobaeza and @chienandalu can you review it?

@Tecnativa

@victoralmau victoralmau changed the title [] [16.0][IMP] dms: Add ACLs to dms.storage for users with the basic permission + Add human_size field to directories and files Feb 19, 2024
@pedrobaeza pedrobaeza added this to the 16.0 milestone Feb 27, 2024
pedrobaeza
pedrobaeza approved these changes Feb 27, 2024
View reviewed changes
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

/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-303-by-pedrobaeza-bump-minor, awaiting test results.

@OCA-git-bot
Copy link
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). 🤖

@OCA-git-bot OCA-git-bot merged commit 31d9d99 into OCA:16.0 Feb 27, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 16.0-imp-dms-fwp-279 branch February 27, 2024 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chienandalu chienandalu chienandalu approved these changes

@pedrobaeza pedrobaeza pedrobaeza approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 ready to merge
Projects
None yet
Milestone
16.0
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@victoralmau @OCA-git-bot @chienandalu @pedrobaeza