-
Notifications
You must be signed in to change notification settings - Fork 277
Home pg new #1038
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
Home pg new #1038
Conversation
cmilesb
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.
A few suggestions for the Cloud portion and general typo/wording stuff.
layouts/home.html
Outdated
| (dict "Text" "Vector search" "URL" "./develop/interact/search-and-query/query/vector-search") | ||
| (dict "Text" "Data structure store" "URL" "./develop/get-started/data-store/") | ||
| (dict "Text" "Document database" "URL" "./develop/get-started/document-database/") | ||
| (dict "Text" "High availability & replication guide" "URL" "./operate/rc/databases/configuration/high-availability/") |
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'm interested to hear why you picked High-Availability for this. I think maybe Create a database might be a better starting link for most people, or maybe the Develop Highly Resilient Apps page? Maybe both?
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 picked this because it is in the top 3 questions people ask about Redis. I think they know they can setup a database, but they want to know if they can do HA in Redis Cloud. I feel like the resilient apps page is something we don't want to really promote at the top page because it is about losing connections when we do maintenance.
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.
Based on the additional feedback, I changed this title to be more general and replaced the HA link with a link to the CE get started page.
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: Cameron Bates <[email protected]>
Co-authored-by: andy-stark-redis <[email protected]>
Co-authored-by: andy-stark-redis <[email protected]>
andy-stark-redis
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.
One tile still looks a bit odd to me, but otherwise looking pretty good now :-)
layouts/home.html
Outdated
| (dict "Text" "Write-behind" "URL" "./integrate/write-behind/quickstart/write-behind-guide/") | ||
| ) | ||
| ) }} | ||
| "Title" "Build modern real-time applications with Redis" |
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.
Although the links in this tile are useful, their common theme seems a bit vague. If you keep the existing title then maybe have some other links (eg, to the quickstart guides), otherwise it looks like the only "modern real-time apps" we support are AI apps. Since this seems to be the only AI tile on the page, though, maybe it's better to change the title to "Build modern AI-enabled apps" or something like that? If you do that, then it's probably best to change the button link to something AI-related too.
If you want a tile that includes Redis Stack install instructions, then maybe add an extra tile to match the ones for Cloud, K8s, etc. Another option might be to change the name of the client libs tile to "Develop your app locally" (or something) and have Install Redis Stack as the button on this tile.
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, I changed this tile to focus on AI and added instead a link to the CE getting started in the tile at the very top.
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.
I think it's looking good so far. I left a couple of comments and questions related to the Redis Software tile.
layouts/home.html
Outdated
| "Title" "Deploy Redis clusters for scale and availability" | ||
| "Description" "Robust support for hybrid and geo-distributed environments" |
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'm not sure about the "for scale and availability" part for Software specifically. Cloud and k8s are also good deployments for scale and availability.
Redis Software gives you more direct control over your deployments since it's self-managed and can be installed on-premises or on cloud platforms.
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 agree that this title is a little vague because all of our products offer scaling and availability.
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, I changed this line to focus on the control part
layouts/home.html
Outdated
| (dict "Text" "Install" "URL" "./operate/oss_and_stack/install/install-stack/") | ||
| (dict "Text" "Redis Query Engine" "URL" "./develop/interact/search-and-query/") | ||
| (dict "Text" "JSON" "URL" "./develop/data-types/json/") | ||
| (dict "Text" "Optimize Redis clusters guide" "URL" "./operate/rs/clusters/optimize") |
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.
What made you choose the optimize cluster section?
Since you mention geo-distributed environments in the description, why not link to the Active-Active docs instead? https://redis.io/docs/latest/operate/rs/databases/active-active/ or maybe https://redis.io/docs/latest/operate/rs/databases/active-active/planning/?
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.
We were also told (granted over a year ago) to stop using "geo-distributed" to describe Active-Active databases. PMs have changed since then, so might not be an issue, but it's worth looking into changing the wording here.
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 changed part of this to multi-AZ
kaitlynmichael
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.
Overall looks good. I would prefer shorter titles, as the ones that wrap are harder to scan. Left some comments on the K8s specific titles.
layouts/home.html
Outdated
| "Title" "Deploy Redis clusters for scale and availability" | ||
| "Description" "Robust support for hybrid and geo-distributed environments" |
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 agree that this title is a little vague because all of our products offer scaling and availability.
layouts/home.html
Outdated
| "ButtonLink" "./operate/rc" | ||
| "ButtonLabel" "Read more" | ||
| "LinksLeftTitle" "Learn more" | ||
| "Title" "Deploy on containers running in Kubernetes" |
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.
| "Title" "Deploy on containers running in Kubernetes" | |
| "Title" "Deploy on Kubernetes" |
I feel like the extra words aren't really necessary and make it harder to scan
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.
fixed
| "ButtonLabel" "Read more" | ||
| "LinksLeftTitle" "Learn more" | ||
| "Title" "Deploy on containers running in Kubernetes" | ||
| "Description" "Get started with the Redis Software for Kubernetes container image" |
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.
| "Description" "Get started with the Redis Software for Kubernetes container image" | |
| "Description" "Get started with the Redis for Kubernetes" |
Container image isn't really the best way to describe this. While K8s does use containers, they are orchestrated by K8s and the user actually interacts with the "deployment" resource, not the container.
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! I changed the button text, but I want to keep the words container and image on this page for SEO for now.
layouts/home.html
Outdated
| (dict "Text" "Install" "URL" "./operate/oss_and_stack/install/install-stack/") | ||
| (dict "Text" "Redis Query Engine" "URL" "./develop/interact/search-and-query/") | ||
| (dict "Text" "JSON" "URL" "./develop/data-types/json/") | ||
| (dict "Text" "Optimize Redis clusters guide" "URL" "./operate/rs/clusters/optimize") |
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.
We were also told (granted over a year ago) to stop using "geo-distributed" to describe Active-Active databases. PMs have changed since then, so might not be an issue, but it's worth looking into changing the wording here.
| "ButtonLink" "./develop/tools/insight/" | ||
| "ButtonLabel" "Learn more" | ||
| "LinksLeftTitle" "Learn more" | ||
| "Title" "Migrate data to or from Redis" |
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.
| "Title" "Migrate data to or from Redis" | |
| "Title" "Migrate data to or from Redis" |
Could we simplify this and say "Migrate data" or "Migrate your data"?
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! I will leave geo-distributed because its still in the title of the doc and I want to be more clear rather than simple with the migrate to or from line here.
andy-stark-redis
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.
Looking pretty good now, from my point of view :-)
cmilesb
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.
Looks good to me.
Modified design with updated text and links https://redis.io/docs/staging/home-pg-new/