Skip to content

RS: Added deprecation note to old module config REST API endpoints #2193

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 30, 2025
Merged

RS: Added deprecation note to old module config REST API endpoints #2193

merged 2 commits into from
Sep 30, 2025
+16 −0

Conversation

rrelledge
Copy link
Collaborator

No description provided.

@rrelledge rrelledge requested a review from a team September 30, 2025 20:44
@rrelledge rrelledge self-assigned this Sep 30, 2025
@rrelledge rrelledge added the rs Redis Enterprise Software label Sep 30, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 30, 2025 •
edited by atlassian bot
Loading

DOC-5782

@github-actions GitHub Actions
Copy link
Contributor

Staging links:
https://redis.io/docs/staging/DOC-5782/operate/rs/7.8/references/rest-api/requests/bdbs/modules/config/
https://redis.io/docs/staging/DOC-5782/operate/rs/7.8/references/rest-api/requests/modules/config/
https://redis.io/docs/staging/DOC-5782/operate/rs/references/rest-api/requests/bdbs/modules/config/
https://redis.io/docs/staging/DOC-5782/operate/rs/references/rest-api/requests/modules/config/

dwdougherty
dwdougherty approved these changes Sep 30, 2025
View reviewed changes
Copy link
Collaborator

@dwdougherty dwdougherty left a comment

Choose a reason for hiding this comment

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

Language LGTM. The only small problem is that there's no whitespace between the page banner and the note section as shown in the attached; looks odd to me, but YMMV.

Screenshot 2025-09-30 at 2 06 06 PM

@rrelledge
Copy link
Collaborator Author

@dwdougherty Oh yeah, I forgot that I had used the banner-article shortcode instead of note in similar situations before. Does this seem less weird?

banner-article

@dwdougherty
Copy link
Collaborator

@dwdougherty Oh yeah, I forgot that I had used the banner-article shortcode instead of note in similar situations before. Does this seem less weird?

Less weird fo' sho'! :)

@rrelledge rrelledge merged commit 91eec93 into main Sep 30, 2025
7 checks passed
@rrelledge rrelledge deleted the DOC-5782 branch September 30, 2025 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dwdougherty dwdougherty dwdougherty approved these changes

Assignees

@rrelledge rrelledge

Labels

rs Redis Enterprise Software

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rrelledge @dwdougherty