Skip to content

Conversation

@dariagrudzien
Copy link
Contributor

Add photos & details of BG Unconference speakers.

I wasn't sure about formatting, so I did my best to get inspired by https://ep2025.europython.eu/speakers/ Please let me know if this can be improved.

@europython-society
Copy link
Member

europython-society commented Jun 25, 2025

Preview available

Key Value
url https://daria-unconf-speakers.ep-preview.click
last update 2025-06-26T09:05:08.756Z

@clytaemnestra
Copy link
Contributor

I think we might have some paths broken, because I don't see any images:

image

@nikoshell
Copy link
Contributor

Add photos & details of BG Unconference speakers.

I wasn't sure about formatting, so I did my best to get inspired by https://ep2025.europython.eu/speakers/ Please let me know if this can be improved.

Pls create directory src/content/pages/beginners-day-unconference
than you can have inside your images and index.mdx

We use mdx to have mardown syntax and to be able to use existing components, we tray to avoid pushing html to mdx instead of that it's better to create components which can be reused in any mdx, if some pages are more complex we just create astro page. I see that in your code you reuse one section multiple times which is perfect example where we should create component.

Feel free to ping me if you will have issue with that, or I can create it for you.

@dariagrudzien dariagrudzien force-pushed the daria/unconf-speakers branch from e22ef49 to 07c2454 Compare June 25, 2025 21:01
@dariagrudzien
Copy link
Contributor Author

@nikoshell Understood. I'd very much appreciate your help, I don't have the mental bandwidth to grok this right now. Feel free to push into this branch.

I created the directory and moved the assets there. Is it okay if I leave it like this for you?

@nikoshell
Copy link
Contributor

@nikoshell Understood. I'd very much appreciate your help, I don't have the mental bandwidth to grok this right now. Feel free to push into this branch.

I created the directory and moved the assets there. Is it okay if I leave it like this for you?

@dariagrudzien I add some changes, and I moved images to /public
if everything is fine we can mege it.

Copy link
Contributor

@clytaemnestra clytaemnestra left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @dariagrudzien for the PR and working on finding speakers, looking forward to it! And thank you @nikoshell for cleaning it up.

@dariagrudzien
Copy link
Contributor Author

@nikoshell This is great, thank you so much I appreciate this a lot!

image

@dariagrudzien dariagrudzien merged commit 317f28b into ep2025 Jun 26, 2025
6 checks passed
@dariagrudzien dariagrudzien deleted the daria/unconf-speakers branch June 26, 2025 09:33
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.

5 participants