-
Notifications
You must be signed in to change notification settings - Fork 113
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
Share manager migration #2911
Share manager migration #2911
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
0d0fc81
to
cc4bf81
Compare
2cf28f9
to
0e647d0
Compare
0e647d0
to
4795230
Compare
Kudos, SonarCloud Quality Gate passed! |
looking good! Is there a CLI to Load and Dump? I don't see where they are used beside the tests... Can you link to it? |
@butonic I have a branch which adds according migration commands to the ocis cli at https://github.com/aduffeck/ocis/commits/migrate-shares |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cli will come in a separate PR
This PR introduces new interfaces for dumpable and loadable share managers and implements dumping shares from the
json
and loading dumps intocs3
manager.