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

[WIP] Sharestorageprovider oc10 sm #1846

Closed
wants to merge 15 commits into from
Closed

[WIP] Sharestorageprovider oc10 sm #1846

wants to merge 15 commits into from

Conversation

aduffeck
Copy link
Contributor

This is WIP to access oc10 shares from ocis.

\cc @butonic

@update-docs
Copy link

update-docs bot commented Jun 30, 2021

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.

@aduffeck aduffeck mentioned this pull request Jul 11, 2021
Closed
@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from 2c98fc9 to cf22fb2 Compare July 12, 2021 07:57
@lgtm-com
Copy link

lgtm-com bot commented Jul 12, 2021

This pull request introduces 2 alerts when merging cf22fb2 into 00cbdaa - view on LGTM.com

new alerts:

  • 2 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jul 12, 2021

This pull request introduces 2 alerts when merging 29cb5a3 into 256d683 - view on LGTM.com

new alerts:

  • 2 for Useless assignment to local variable

@labkode
Copy link
Member

labkode commented Jul 12, 2021

@aduffeck this looks great. With this PR we could mount the shares under /Shares rather than /home/Shares and simplify many code and make the sync client work out of the box to sync shares.

You have some background here: owncloud/product#7

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2021

This pull request introduces 1 alert when merging dc917cd into 4cf9aac - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@aduffeck aduffeck mentioned this pull request Jul 13, 2021
Closed
@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2021

This pull request introduces 1 alert when merging 87ea2bb into a87aa93 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2021

This pull request introduces 1 alert when merging 6cab9e9 into 089745e - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jul 14, 2021

This pull request introduces 1 alert when merging e487d12 into 089745e - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from e487d12 to 6107e85 Compare July 15, 2021 07:58
@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2021

This pull request introduces 1 alert when merging 6107e85 into 73a4c55 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from 48d729b to 356cd49 Compare July 16, 2021 07:27
@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2021

This pull request introduces 1 alert when merging 356cd49 into 7df477f - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2021

This pull request introduces 1 alert when merging 11be043 into 7df477f - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from 35b26e8 to 88e1544 Compare July 19, 2021 10:05
@lgtm-com
Copy link

lgtm-com bot commented Jul 19, 2021

This pull request introduces 2 alerts when merging 88e1544 into 7df477f - view on LGTM.com

new alerts:

  • 2 for Useless assignment to local variable

@lgtm-com
Copy link

lgtm-com bot commented Jul 20, 2021

This pull request introduces 2 alerts when merging 9aa52d6 into 7df477f - view on LGTM.com

new alerts:

  • 2 for Useless assignment to local variable

@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from 937208f to 044bb2c Compare July 22, 2021 08:03
@lgtm-com
Copy link

lgtm-com bot commented Jul 22, 2021

This pull request introduces 1 alert when merging 044bb2c into 1fb0bd6 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from 2d17347 to 446a759 Compare July 22, 2021 08:41
@lgtm-com
Copy link

lgtm-com bot commented Jul 22, 2021

This pull request introduces 1 alert when merging a1bef23 into 1cbc542 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch 4 times, most recently from d84ab5b to 9c30844 Compare August 3, 2021 13:51
@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch 2 times, most recently from d43f7cc to 34052cf Compare August 6, 2021 08:27
@aduffeck
Add a sharesstorageprovider
2a77248 
The provider exposes all received shares. It can be mounted to /home/Shares
to make a lot of special cases for shares handling in the gateway and
storage drivers superfluous.
@aduffeck aduffeck force-pushed the sharestorageprovider-oc10-sm branch from 34052cf to 29327c2 Compare August 17, 2021 12:15
@butonic butonic mentioned this pull request Aug 17, 2021
Open
2 tasks
@wkloucek wkloucek mentioned this pull request Aug 30, 2021
Merged
9 tasks
@butonic butonic mentioned this pull request Aug 31, 2021
Closed
3 tasks
@butonic
Copy link
Contributor

butonic commented Aug 31, 2021

continuing in #2023

@butonic butonic closed this Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@labkode labkode Awaiting requested review from labkode labkode will be requested when the pull request is marked ready for review labkode is a code owner

@glpatcern glpatcern Awaiting requested review from glpatcern glpatcern will be requested when the pull request is marked ready for review glpatcern is a code owner

@diocas diocas Awaiting requested review from diocas diocas will be requested when the pull request is marked ready for review diocas is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@aduffeck @labkode @butonic