[LJU-2025] Ljubljana 25 kickoff by nimatel · Pull Request #15165 · devopsdays/devopsdays-web

nimatel · 2025-05-10T12:54:42Z

Ljubljana 25 kick-off

used script to generate Ljubljana event
copied logo, modified welcome page

netlify · 2025-05-10T12:54:47Z

✅ Deploy Preview for devopsdays-web ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	`5e1f589`
🔍 Latest deploy log	https://app.netlify.com/projects/devopsdays-web/deploys/6827c28a4d2e94000825215a
😎 Deploy Preview	https://deploy-preview-15165--devopsdays-web.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

toshywoshy · 2025-05-10T15:38:23Z

static/events/2025-ljubljana/logo.png

Please move this to the assets folder
assets/events/2025-ljubljana/logo.png

static/events/2025-ljubljana/logo-square.jpg

data/events/2025/ljubljana/main.yml

toshywoshy · 2025-05-10T15:45:21Z

content/events/2025-ljubljana/welcome.md

+        <strong>Speakers:</strong>
+      </div>
+      <div class = "col-md-8">
+        Do you have knowledge to share? Apply <a href="http://cfp.devopsdays.si/" target="_blank">here</a>


This does not seem live for now
As indicated in the next remark, please use the shortcode, to avoid out of sync links
{{< event_link url-key="cfp_link" text="here" >}}

This will be live at a flip of a switch - please bare in mind the date. Also, we wanted to get the org site first, then the webpage and cfp

The site is live, and the use of the shortcode isn't a blocker as much as a helpful practice.

Just out of curiosity, though, did you intend to use http:// instead of https:// here?

data/events/2025/ljubljana/main.yml

nimbinatus

There's a few things (the social links mainly, and the logo move to assets) that need to be addressed. Most of my comments are more FYI and not a blocker for merge (I've tried to mark all blockers clearly for you).

nimbinatus · 2025-05-16T15:16:17Z

content/events/2025-ljubljana/propose.md

@@ -0,0 +1,35 @@
+++


As a note (which I'll repeat in the data file), you can delete this file and make the link at the top of your welcome page point directly to a CFP page. Just so someone doesn't stumble across this page since it's still getting built and then email your list with a proposal.

Not a blocker! Just an FYI.

nimbinatus · 2025-05-16T15:16:58Z

content/events/2025-ljubljana/registration.md

@@ -0,0 +1,11 @@
+++


Same here as for the proposal page :)

Not a blocker; just an FYI.

nimbinatus · 2025-05-16T15:18:04Z

content/events/2025-ljubljana/welcome.md

+        <strong>Speakers:</strong>
+      </div>
+      <div class = "col-md-8">
+        Do you have knowledge to share? Apply <a href="http://cfp.devopsdays.si/" target="_blank">here</a>


The site is live, and the use of the shortcode isn't a blocker as much as a helpful practice.

Just out of curiosity, though, did you intend to use http:// instead of https:// here?

nimbinatus · 2025-05-16T15:18:31Z

content/events/2025-ljubljana/welcome.md

+        <strong>Sponsors:</strong>
+      </div>
+      <div class = "col-md-9">
+        Position your brand. Learn about our sponsorship packages


Did you intend to have a link to the sponsor page here?

we have not finalized how sponsorship packages will look like so it's not linked - yet

data/events/2025/ljubljana/main.yml

nimbinatus · 2025-05-16T15:24:57Z

data/events/2025/ljubljana/main.yml

+event_social_twitter: "devopsdays" # Change this to the twitter handle for your event such as devopsdayschi or devopsdaysmsp
+event_social_mastadon: "https://mastodon.social/@devopsdays" # Change this to url to your mastadon page
+event_social_bsky: "https://bsky.app/profile/devopsdays.bsky.social" # Change this to url to your bluesky page
+event_social_youtube: "devopsdays" # Change this to the youtube channel handle for your event such as devopsdaysrox


Blocker: Please change this to your city's page or comment out.

nimbinatus · 2025-05-16T15:25:04Z

data/events/2025/ljubljana/main.yml

+event_social_bsky: "https://bsky.app/profile/devopsdays.bsky.social" # Change this to url to your bluesky page
+event_social_youtube: "devopsdays" # Change this to the youtube channel handle for your event such as devopsdaysrox
+# legacy
+event_twitter: "devopsdays" # This will create a traditional "Follow" twitter button. Change this to the twitter handle for your event such as devopsdayschi or devopsdaysmsp.


Not the right link; please update or comment out

nimbinatus · 2025-05-16T15:26:52Z

data/events/2025/ljubljana/main.yml

+event_twitter: "devopsdays" # This will create a traditional "Follow" twitter button. Change this to the twitter handle for your event such as devopsdayschi or devopsdaysmsp.
+
+nav_elements: # List of pages you want to show up in the navigation of your page.
+ # - name: propose


If you'd like to change the link in the header and then remove the markdown for the page as I mentioned in a previous comment, here's how:

Suggested change

# - name: propose

- name: propose

url: <insert-url-here>

Not a blocker; just an FYI.

data/events/2025/ljubljana/main.yml

nimbinatus · 2025-05-16T15:32:54Z

static/events/2025-ljubljana/logo-square.jpg

If you check the preview, the logo isn't showing up on the main page: https://deploy-preview-15165--devopsdays-web.netlify.app/

Please copy this file to assets/events/2025-ljubljana to have it appear properly on the main page.

nimbinatus

Other than the logo broken on the welcome page (reference the review), everything looks great. Since the content is there, I'm merging. Please open a PR to patch the logo display on your welcome page when you can.

nimbinatus · 2025-05-18T09:31:46Z

content/events/2025-ljubljana/welcome.md

+
+<div class="row" style="background-color: #f8f9fa; padding: 20px; border-radius: 8px; box-shadow: 0 4px 8px rgba(0, 0, 0, 0.1);">
+  <div class="col-md-4" style="text-align: center;">
+    <img alt="DevOpsDays Ljubljana 2025" src="/events/2025-ljubljana/logo-square.jpg" class="img-fluid" style="max-width: 100%; border-radius: 8px;">


Note that this image isn't displaying with the move to assets. The short code Tosh mentioned is what you can use here to have it display. I'm reviewing on mobile, so I apologize that I don't have the link to how to use it handy. I know you all need this merged soon, so I'm approving and merging. Open a PR with the patch for getting the logo displaying, and we'll get it merged, too

Stale

Ljubljana 25 kickoff

b3c1465

nimatel requested a review from a team as a code owner May 10, 2025 12:54

Merge branch 'main' into lju-2025

3883fd7

toshywoshy previously requested changes May 10, 2025

View reviewed changes

nimatel added 5 commits May 11, 2025 22:04

Update the PR request

d7248e7

Typo Fix

5661f72

Merge branch 'main' into lju-2025

db2940f

Typo fix

3d43e68

Update main.yml

d15dea8

nimatel requested a review from toshywoshy May 11, 2025 20:23

nimatel and others added 4 commits May 13, 2025 08:03

Merge branch 'main' into lju-2025

acc64a8

Merge branch 'main' into lju-2025

ce7b346

Merge branch 'main' into lju-2025

f405974

Merge branch 'main' into lju-2025

04e7ca3

toshywoshy mentioned this pull request May 15, 2025

[LJUBLJANA-2025] Kick off update #15184

Closed

nimbinatus reviewed May 16, 2025

View reviewed changes

nimbinatus added the changes-requested Changes have been requested, and are pending updates. label May 16, 2025

nimatel added 2 commits May 17, 2025 00:40

Update values based on open PR

ba4c4cf

Merge remote-tracking branch 'origin/lju-2025' into lju-2025

c2bdbd6

nimatel requested a review from nimbinatus May 16, 2025 22:47

Merge branch 'main' into lju-2025

5e1f589

nimbinatus approved these changes May 18, 2025

View reviewed changes

nimbinatus merged commit fc02cd8 into devopsdays:main May 18, 2025
8 checks passed

Conversation

nimatel commented May 10, 2025

Uh oh!

netlify bot commented May 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for devopsdays-web ready!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

nimbinatus left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nimbinatus left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

netlify bot commented May 10, 2025 •

edited

Loading

nimbinatus left a comment •

edited

Loading