-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: Better link language SDKs #6866
Conversation
The SDK support is there for many languages now and these are likely a preferred way to use OPA's REST API for many application developers. This PR adds some links to make sure these are featured in suitable locations so they are not missed as people browse the docs. Signed-off-by: Charlie Egan <charlie@styra.com>
✅ Deploy Preview for openpolicyagent ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
docs/content/integration.md
Outdated
When your application or service needs to make policy decisions it can query OPA | ||
locally via HTTP. While it's possible to call OPA's [REST API](../rest-api) directly, | ||
you can also find a number of [native language REST SDKs](/ecosystem/#languages) | ||
eoo which make the integration easier. |
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 has a weird typo in it -- "EEO"
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.
😬 not sure what that one was meant to be...
docs/content/rest-api.md
Outdated
OPA integration using one of the [language SDKs](/ecosystem/#languages) than working | ||
with the REST API directly. | ||
{{< /info >}} | ||
|
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.
I find this note a bit misplaced. I mean we're on the rest api page and right at the top we're hinting that this approach may not be ideal. I would rather have this somewhere in the text later on or at the end imo.
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.
The point is to ensure that users are aware that they dont have to deal with the REST API unless they need to.
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.
I've opted to move this down to the end of the intro, it still features prominently above the fold.
Signed-off-by: Charlie Egan <charlie@styra.com>
The SDK support is there for many languages now and these are likely a preferred way to use OPA's REST API for many application developers.
This PR adds some links to make sure these are featured in suitable locations so they are not missed as people browse the docs.