-
Notifications
You must be signed in to change notification settings - Fork 277
RED-174158: Add Redis Cluster API in docs #2418
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
base: main
Are you sure you want to change the base?
Conversation
|
Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset. In case there are security findings, they will be communicated to you as a comment inside the PR. Hope you’ll enjoy using Jit. Questions? Comments? Want to learn more? Get in touch with us. |
|
|
| <!-- --> | ||
| <!-- --> | ||
| <<TEST>> |
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 imagine this was added mistakenly
paoloredis
left a comment
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.
Rest looks good!
rrelledge
left a comment
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.
Thanks, @sonya-spasova! This looks great so far. I have a couple of initial questions and requested changes.
Can you move this API reference location underneath the existing API reference directory (https://redis.io/docs/latest/operate/rs/references/rest-api/)?
Ideally, I would like to eventually remove https://redis.io/docs/latest/operate/rs/references/rest-api/requests/ and https://redis.io/docs/latest/operate/rs/references/rest-api/objects/ and redirect readers to the new API reference at https://redis.io/docs/latest/operate/rs/references/rest-api/api-reference instead of the current URL https://redis.io/docs/latest/operate/rs/api/api-reference/.
| }, | ||
| "servers": [ | ||
| { | ||
| "url": "http://localhost:3346" |
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.
Is 3346 an internal API port? We've been using examples like https://localhost:9443/v1/users in our public docs.
No description provided.