-
Notifications
You must be signed in to change notification settings - Fork 369
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
feat: develop Tetragon Enterprise page #3103
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for tetragon ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
228b809
to
6076ca6
Compare
docs/content/en/enterprise.html
Outdated
<a class="social__link" href="https://isovalent.com/labs/tetragon-getting-started/" target="_blank" | ||
rel="noopener noreferrer"> | ||
<img src="/svgs/social/lab-icon.svg" width="64" height="64" loading="lazy" alt="" /> | ||
<span>Explore Tetragon lab</span> |
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.
<span>Explore Tetragon lab</span> | |
<span>Explore Tetragon labs</span> |
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
6076ca6
to
8185e1a
Compare
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.
Technically looks good, not sure repeating the community thing in the enterprise is important but if it's what was decided I have no strong feeling.
If we can avoid copy pasting as well that would be great.
docs/content/en/_index.html
Outdated
</li> | ||
</ul> | ||
</div> | ||
<h2 class="title social__title">Welcome to the community!</h2> |
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.
you could use a shortcode to reuse that in multiple pages instead of copy pasting
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.
Done - used a shortcode for the social section
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.
let's wait for an ack from @xmulligan
8185e1a
to
531a5b1
Compare
Signed-off-by: Anna Artemova <[email protected]>
531a5b1
to
937a8d4
Compare
Describe what changes this pull request brings
Welcome to the community!
sectionSteps to check: