Skip to content

Conversation

LORDBABUINO
Copy link
Contributor

Removes Organizers

  • Ana Sabino
  • Matheus Araujo
  • Rafael Santos
  • Guaracy Carvajal
  • Gustavo Santos

Adds event schedule

@LORDBABUINO LORDBABUINO requested a review from a team as a code owner November 22, 2024 06:20
Copy link

netlify bot commented Nov 22, 2024

Deploy Preview for devopsdays-web ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 5d2039e
🔍 Latest deploy log https://app.netlify.com/sites/devopsdays-web/deploys/67a3a78698f53a0008e17140
😎 Deploy Preview https://deploy-preview-14740--devopsdays-web.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.

@toshywoshy toshywoshy added the awaiting-email-to-info An email to info@ is pending. label Nov 22, 2024
@LORDBABUINO LORDBABUINO requested a review from a team as a code owner November 22, 2024 20:36
Copy link
Member

@nimbinatus nimbinatus left a comment

Choose a reason for hiding this comment

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

The global core updates are complete; @toshywoshy, I'm not seeing the inconsistency on the two pages? Since this is past, though, we should merge it.

@nimbinatus nimbinatus removed the awaiting-email-to-info An email to info@ is pending. label Jan 14, 2025
Copy link
Member

@nimbinatus nimbinatus left a comment

Choose a reason for hiding this comment

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

Let's move theme changes to a different PR.

@@ -0,0 +1,243 @@
{{- $.Scratch.Set "first" "true" -}}
Copy link
Member

@nimbinatus nimbinatus Jan 16, 2025

Choose a reason for hiding this comment

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

Sorry, just noticed this. Why are you changing the theme here in this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just made the program be loaded from a partial and the title load dynamically. Should I do it in another PR?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, please separate content and code changes so that the code changes can be reviewed (and not block your content updates!)

@nimbinatus nimbinatus merged commit a0f4af9 into devopsdays:main Feb 6, 2025
8 checks passed
@nimbinatus
Copy link
Member

Thanks @LORDBABUINO! Looking forward to the code changes in a different PR :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants