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

[AMS-2025] Add Program and Speakers #14935

Merged
merged 2 commits into from
Mar 3, 2025
Merged

Conversation

timothymamo
Copy link
Contributor

As per title.

@timothymamo timothymamo requested a review from a team as a code owner March 3, 2025 09:07
Copy link

netlify bot commented Mar 3, 2025

👷 Deploy Preview for devopsdays-web processing.

Name Link
🔨 Latest commit 0734204
🔍 Latest deploy log https://app.netlify.com/sites/devopsdays-web/deploys/67c571634dba60000863ca84
Start building the best web experiences in record time

Copy link

netlify bot commented Mar 3, 2025

Deploy Preview for devopsdays-web ready!

Built without sensitive environment variables

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

Copy link
Contributor

@toshywoshy toshywoshy left a comment

Choose a reason for hiding this comment

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

Why are you embedding?
The preferred way is to use the program as per the documentation or use the conferencehook to create a DoD program

please <a target="_blank" href="https://talks.devopsdays.org/devopsdays-amsterdam-2025/schedule/">click here</a>.
</div>
</div>
</noscript>
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an embedding of the schedule, and not an addition

.finally(() => {
ul.appendChild(list);
});
</script>
Copy link
Contributor

Choose a reason for hiding this comment

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

Again this is an embedding of the schedule, not an addition

@timothymamo
Copy link
Contributor Author

timothymamo commented Mar 3, 2025

Why are you embedding? The preferred way is to use the program as per the documentation or use the conferencehook to create a DoD program

@toshywoshy Could you link to the documentation. Also Austin 2024 page as proposed by @nimbinatus.

@toshywoshy
Copy link
Contributor

The documentation only talks about have the program on the DoD website, this is the normal way of working.
@nimbinatus experimenten with the embedded program, because more control was wanted, with the idea that the content would be copied into the DoD website, which as you can see never happened.
There is a reason we do not have a shortcode for this, as this is not a preferred way of working
If you do not want to create a program, you can generate one using the conference hook

@timothymamo
Copy link
Contributor Author

I have raised this within the DevOpsDays slack channel. Note: that after the conference it was discussed within the Amsterdam org team that we would convert the pages to use the regular format (bunch of yaml) for prosperity.

@timothymamo
Copy link
Contributor Author

As requested:

The benefits of publishing it this way means that changes to the schedule do need to be managed by the website maintainers and also Open Spaces can be updated on the fly during the conference for the benefit of the attendees.

@yvovandoorn has also advertised the benefits of using the Pretalx widget recently.

@toshywoshy
Copy link
Contributor

At the speed we currently process PR, changes to schedule end up taking very limited time.
In the past there were less reviewers and things did take longer, however unless you need to mail info or get core approval, changes are quick and follow the sun, there are enough of us, asleep and awake merging things in.

As for what @yvovandoorn advertised, this is an option, however we have preferred not to encourage this option and prefer that events use the program on the DevOpsDays website.

@timothymamo
Copy link
Contributor Author

The Amsterdam organising team has discussed this and agreed to post the schedule in this method. If there is an official reason which the organization team has to comply with I will change the PR.

Otherwise please merge.

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.

Approving as website lead. Discussion happening elsewhere.

@nimbinatus nimbinatus dismissed toshywoshy’s stale review March 3, 2025 15:21

Discussion happening elsewhere. Unblocking.

@nimbinatus nimbinatus merged commit 8fdc02c into devopsdays:main Mar 3, 2025
8 checks passed
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.

3 participants