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][ADD] dms_storage #275

Merged
merged 4 commits into from
May 20, 2024
Merged

Conversation

len-foss
Copy link
Contributor

Migration of #182 depending on fs_storage instead of storage_backend.

(The same TODO applies, which is that changing the storage of existing directories is not supported yet)

@len-foss
Copy link
Contributor Author

Tests should work once OCA/storage#314 (or maybe another one if it ends being split) gets merged.

Copy link

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 Apr 14, 2024
@github-actions github-actions bot closed this May 19, 2024
@pedrobaeza pedrobaeza reopened this May 19, 2024
@pedrobaeza pedrobaeza removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 19, 2024
@pedrobaeza
Copy link
Member

Please rebase

@len-foss len-foss force-pushed the 16.0-dmsstorage-len branch from 1159440 to 754c421 Compare May 19, 2024 17:56
@len-foss
Copy link
Contributor Author

Done :-)

@pedrobaeza pedrobaeza added this to the 16.0 milestone May 20, 2024
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.

Let's merge this useful one:

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 16.0-ocabot-merge-pr-275-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit e5e4da1 into OCA:16.0 May 20, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at db3d259. 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.

4 participants