Skip to content

[PT-2026] Fix images at welcome page and add kcd Porto and DevOps Porto as community sponsors#15914

Open
somatorio wants to merge 3 commits into
devopsdays:mainfrom
devopsdaysportugal:main
Open

[PT-2026] Fix images at welcome page and add kcd Porto and DevOps Porto as community sponsors#15914
somatorio wants to merge 3 commits into
devopsdays:mainfrom
devopsdaysportugal:main

Conversation

@somatorio
Copy link
Copy Markdown
Contributor

Note: This includes a new shortcode to the theme

@somatorio somatorio requested review from a team as code owners May 20, 2026 11:43
@netlify
Copy link
Copy Markdown

netlify Bot commented May 20, 2026

Deploy Preview for devopsdays-web ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 0d85ad0
🔍 Latest deploy log https://app.netlify.com/projects/devopsdays-web/deploys/6a0dacaa77bf0e0009064d39
😎 Deploy Preview https://deploy-preview-15914--devopsdays-web.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@phrawzty
Copy link
Copy Markdown
Member

Hello @somatorio,

The parts of this PR that touch only the PT-2026 event are reasonable; however, this PR also modifies the theme, which is a different request altogether. It will need to go through a more rigorous review by the web team, and to be blunt: that might take a while.

My strong suggestion: make two PRs. One that gets you the visual result you want using the existing theme, and then another that proposes the theme modification for RFC.

Let me know how you want to proceed.

@somatorio
Copy link
Copy Markdown
Contributor Author

To make that change without the shortcode I would have to duplicate the kcd-porto logo, having it at the assets and a copy at static :/

Unless there is another way and @toshywoshy can point me towards it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants