-
Notifications
You must be signed in to change notification settings - Fork 5
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
Create deployment options page #543
Create deployment options page #543
Conversation
Visit the preview URL for this PR (updated for commit 9e9b245): https://estuary-marketing--pr543-brenosalv-feature-51-9xdv0uvu.web.app (expires Sun, 24 Nov 2024 06:27:03 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 76f6b095a0752e5d9c6c890267f9fdc3e392161e |
…to Brenosalv/feature/513-deployment-options-page
…to Brenosalv/feature/513-deployment-options-page
@@ -0,0 +1,23 @@ | |||
export const features = { |
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.
Instead of having a stand alone list in the type DeploymentOption
we should just reference this. That way it will always be in sync.
export type DeploymentOption = keyof typeof features;
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.
Makes sense, thanks.
<Seo | ||
title="Estuary Flow Deployment Options: Public, Private & BYOC" | ||
description="Estuary Flow offers Public, Private, and BYOC deployment options for secure, scalable, and customizable data integration, perfect for any business needs." | ||
image={metaImg.childImageSharp.gatsbyImageData.images.fallback.src} |
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.
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 small change to typing
I created a container component and applied it to all the sections. I still need to do this for sections of the other pages. |
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
#513
Changes
The path/slug is
/deployment-options
.