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

Add support for Spaces #4404

Open
wants to merge 141 commits into
base: master
Choose a base branch
from
Open

Add support for Spaces #4404

wants to merge 141 commits into from

Conversation

glpatcern
Copy link
Member

@glpatcern glpatcern commented Dec 13, 2023 •
edited
Loading

All credits to @gmgigi96, I reopened this PR so we can keep working on it.

Missing parts:

  • Expose permission to share (currently, the frontend believes the user can't share anything and hides the "shared by me" view)
  • OCS API to support the new storageid format
  • Trashbin support
  • Share hiding/accepting
    A way to enable/disable this format by config, such that we can have the functionality in the code and enable it once ready

And from the last merge:

  • Look into all replaces in go.mod, in particular for the cs3apis

@gmgigi96
removed unused config variable
2bf2c87
@gmgigi96
removed old code of spaces
cce7bbc
@gmgigi96
add boilerplate for spaces
04a4243
@gmgigi96
add literals for space types
64602c6
@gmgigi96
add memory driver for spaces
adb7778
@gmgigi96
removed sql file
f9dcee2
@gmgigi96
add spaces loader
82c6f51
@gmgigi96
load spaces manager drivers
0caea57
@gmgigi96
add function to get spaces client
3411bf0
@gmgigi96
redirect spaces request to spaces server
1f6e508
@gmgigi96
add ocgraph service
025f6ab
@gmgigi96
add graph spec
505c67d
@gmgigi96
add permissions in the response
c451ac5
@gmgigi96
read and parse graph api filters to cs3 filters
fb5b43f
@gmgigi96
add filters when getting spaces
319b9eb
@gmgigi96
temporary use my fork for go-cs3apis
8616525
@gmgigi96
fixes
464b42a
@gmgigi96
load spacesregistry grpc service
de140a7
@gmgigi96
require user_space field in memory spaces driver
b066815
@gmgigi96
fix urls
b5f1788
@gmgigi96
fix permissions
bd5467e
@gmgigi96
fix role names
32132e4
@gmgigi96
fixes in permissions
fa2ee5f
@gmgigi96
add missing fields
6df8b37
@gmgigi96
use path as drive alias
e83bada
@gmgigi96
add get me endpoint
014070d
@gmgigi96
fix pattern
2d085e9
@gmgigi96
mock api endpoint
6380bec
@gmgigi96
register ocapi svc
7bee0a1
@gmgigi96
make all api unprotected
ca1e813
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 is a code owner

@gmgigi96 gmgigi96 Awaiting requested review from gmgigi96

@diocas diocas Awaiting requested review from diocas diocas is a code owner

At least 1 approving review is required to merge this pull request.

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
@glpatcern @gmgigi96 @labkode