-
Notifications
You must be signed in to change notification settings - Fork 270
RS: Added rladmin commands to change node roles #2144
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
Conversation
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.
LGTM.
| 1. On the **Nodes** screen, click the **More actions** button (**⋮**) for the primary node you want to demote. | ||
|
|
||
| {{<image filename="images/rs/screenshots/nodes/primary-node-more-actions.png" alt="Click the more actions button for a node to access node actions.">}} | ||
| <img src="../../../../images/rs/screenshots/nodes/primary-node-more-actions.png" alt="Click the more actions button for a node to access node actions."> |
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.
Just because I'm curious: why wouldn't you use the full path ( /images/rs/... ) instead of a relative path like above?
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.
@dwdougherty Thanks for reviewing!
I tried using something like <img src="/images/rs/... at first, but the images only rendered properly in my local env. When I checked the staged preview, the images were actually broken. It would try to use https://redis.io/images/rs/screenshots/... instead of the correct URL https://redis.io/docs/staging/DOC-####/images/rs/screenshots/....
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 should have know that you'd have tried the full path. 😃
Also cherry-picked the img fix/workaround that allows screenshots within the multitabs shortcode.