-
Notifications
You must be signed in to change notification settings - Fork 222
Move redisvl documentation #1770
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
@mich-elle-luna Should https://redis.io/docs/staging/DOC-5395/integrate/redisvl/ just have a link pointing to the other page? https://redis.io/docs/staging/DOC-5395/integrate/redisvl/ |
Yes, I think having just a link pointing to the other page is fine. Also, I think all the relrefs on other pages that point to the redisVL docs should also be updated to reflect the new location as well as part of this PR. Maybe there aren't many? But, I should see at least one in this PR I think. Thank you! |
Good point, I fixed the relrefs and I added a link in https://redis.io/docs/staging/DOC-5395/integrate/redisvl/ . Alternatively, we could also add aliases in https://redis.io/docs/staging/DOC-5395/develop/ai/redisvl/ . |
Thanks! Yeah, I think we have to do both the relrefs and aliases when we move pages in this hugo site (aliases for folks who may have bookmarked before and relrefs to keep existing internal links working). |
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.
Thank you! I think I created a conflict with my big merge just now, but otherwise this looks good to me.
No description provided.