-
Notifications
You must be signed in to change notification settings - Fork 189
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
[docs-only] update ocm service readme #9814
Conversation
cab4fd3
to
205ccc1
Compare
@@ -57,8 +127,12 @@ The data backend of the `ocminvitemanager` is configurable. The only supported b | |||
|
|||
## Creating Shares | |||
|
|||
{{< hint info >}} | |||
The below info is outdated as we allow creating federated shares using the graph API. Clients can now discover the available sharing roles and invite federated users using the graph API. | |||
{{< /hint >}} |
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.
This needs a rework:
- link to libre graph api docs
- hm use existing examples from there?
- prose
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.
Looking good, just a typo :)
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
Co-authored-by: Phil Davis <[email protected]>
7ab90a8
to
be026cd
Compare
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.
As discussed with @butonic LGTM 👍
Quality Gate passedIssues Measures |
[docs-only] update ocm service readme
I tried to make the setup and what goes inside the ocmproviders file clearer.