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

initial namespaces.md #499

Closed
wants to merge 1 commit into from
Closed

Conversation

butonic
Copy link
Member

@butonic butonic commented Aug 27, 2020

No description provided.

@update-docs
Copy link

update-docs bot commented Aug 27, 2020

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.

@butonic butonic marked this pull request as draft August 27, 2020 09:09
@butonic butonic mentioned this pull request Aug 27, 2020
1 task
Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

See comments.

docs/namespaces.md Outdated Show resolved Hide resolved
docs/namespaces.md Outdated Show resolved Hide resolved
docs/namespaces.md Outdated Show resolved Hide resolved
docs/namespaces.md Outdated Show resolved Hide resolved
docs/namespaces.md Outdated Show resolved Hide resolved
docs/namespaces.md Outdated Show resolved Hide resolved
docs/namespaces.md Outdated Show resolved Hide resolved
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic butonic force-pushed the document-namespaces branch from 0c2d155 to 61cf74e Compare August 27, 2020 11:57
@butonic butonic requested a review from PVince81 August 27, 2020 12:33
Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

looks better.
a few more tweaks in the table were nice


Before we can dig into the namespaces we should have a look at the services that
For WebDAV requests:

Copy link
Contributor

Choose a reason for hiding this comment

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

let's call ocdav "reva-frontend (ocdav)" if these are service names and maybe also "ocis-proxy" to clarify further.
currently the naming of the services is mixed



| scenario | endpoint | relative path | relevant ocdav config | cs3 namespace | storage_provider_config | user layout example | internal path |
|----------------------------------------------------------------------------------------|-------------------------------------------------------------------------------------|-----------------------------|---------------------------------------------------------------------|----------------------------------------------------------------------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|-------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------------------------------------------------|
Copy link
Contributor

Choose a reason for hiding this comment

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

add bullets or stars in front of the config settings, to make them more readable

@butonic
Copy link
Member Author

butonic commented Jul 1, 2021

needs a refresh.

@butonic butonic self-assigned this Jul 1, 2021
@micbar
Copy link
Contributor

micbar commented Nov 29, 2021

@butonic closing because outdated.

@micbar micbar closed this Nov 29, 2021
@butonic butonic deleted the document-namespaces branch June 8, 2022 11:11
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.

3 participants