Skip to content

SITE: fix broken links #1520

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

Merged
merged 5 commits into from
May 9, 2025
Merged

SITE: fix broken links #1520

merged 5 commits into from
May 9, 2025

Conversation

dwdougherty
Copy link
Collaborator

@dwdougherty dwdougherty commented May 7, 2025

DOC-5212

I fixed all but the RedisVL content. All the issues with the "modules.html" embed are solved by removing that file; it's a remnant from the old .com site and it's no longer needed (confirmed by @rrelledge).

I'm leaving row 9 (RedisVL) for @paoloredis; it seems that content is missing from the port automation. Not sure though. I didn't want to make any changes that will just be overwritten next time the automation runs.

@dwdougherty dwdougherty requested review from mich-elle-luna and a team May 7, 2025 21:12
@dwdougherty dwdougherty added the bug Something isn't working label May 7, 2025
Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

One actual glitch (I think) and a suggestion, but otherwise LGTM.

@dwdougherty
Copy link
Collaborator Author

Thank you, @andy-stark-redis!

@dwdougherty
Copy link
Collaborator Author

Since @kaitlynmichael is out today (5/8/2025), I went ahead and took a shot at updating her section of the broken links sheet.

@dwdougherty dwdougherty requested a review from kaitlynmichael May 8, 2025 14:31
Copy link
Collaborator

@mich-elle-luna mich-elle-luna left a comment

Choose a reason for hiding this comment

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

thank you!

@dwdougherty dwdougherty merged commit 4b1ff1c into main May 9, 2025
5 checks passed
@dwdougherty dwdougherty deleted the DOC-5212 branch May 9, 2025 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working do not merge yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants