Skip to content
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: add banner to community meetings and move tetragon slides #2801

Merged
merged 1 commit into from
Aug 16, 2024

Conversation

annaindistress
Copy link
Contributor

Description

Add banner before header and move slider section after community links at the homepage

image image

@annaindistress annaindistress requested review from mtardy and a team as code owners August 15, 2024 12:40
Copy link

netlify bot commented Aug 15, 2024

Deploy Preview for tetragon ready!

Name Link
🔨 Latest commit 3954ec0
🔍 Latest deploy log https://app.netlify.com/sites/tetragon/deploys/66bf609d1179930008f1eba0
😎 Deploy Preview https://deploy-preview-2801--tetragon.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@paularah
Copy link
Contributor

@annaindistress can you request a review from me?

@annaindistress
Copy link
Contributor Author

@annaindistress can you request a review from me?

It seems that I'm unable to select reviewers

image

@lambdanis lambdanis added the release-note/docs This PR updates the documentation. label Aug 15, 2024
@lambdanis
Copy link
Contributor

@paularah Done. FYI, only Cilium GH org members can assign reviewers.

@paularah
Copy link
Contributor

@paularah Done. FYI, only Cilium GH org members can assign reviewers.

oopsie. thanks

Copy link
Contributor

@paularah paularah left a comment

Choose a reason for hiding this comment

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

LGTM

docs/content/en/_index.html Outdated Show resolved Hide resolved
Copy link
Member

@mtardy mtardy left a comment

Choose a reason for hiding this comment

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

Thanks! looks good like that :)

@mtardy
Copy link
Member

mtardy commented Aug 16, 2024

Ah you might have to rebase the LinkedIn PR sorry I forgot to look at it after you pushed changes. Let's merge this one as it's bigger.

@mtardy mtardy merged commit 6e4101e into cilium:main Aug 16, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/docs This PR updates the documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants