Add section "The Week" mini version #1301

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.

Sign up for GitHub

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

Jump to bottom

Closed

nikoshell wants to merge 2 commits into ep2025 from ep2025-the-week

Contributor

nikoshell commented Jun 9, 2025

Close #1292

nikoshell added 2 commits

June 9, 2025 10:53


          Add section


          Merge branch 'ep2025' into ep2025-the-week

f850696

Member

europython-society commented Jun 9, 2025 •

edited

Loading

Preview available

Key	Value
url	https://ep2025-the-week.ep-preview.click
last update	2025-06-09T08:58:44.090Z

nikoshell self-assigned this

nikoshell requested a review from clytaemnestra

June 9, 2025 09:09

hugovk reviewed

View reviewed changes

src/components/sections/theweek.astro

+              const weekPeriods: WeekPeriod[] = [
+                {
+                  id: 'monday-tuesday',
+                  days: 'Mon – Tues',

Member

hugovk Jun 9, 2025

Suggested change

      
                days: 'Mon – Tues',
          
                days: 'Monday – Tuesday',

src/components/sections/theweek.astro

+                },
+                {
+                  id: 'wednesday-friday',
+                  days: 'Wed – Friday',

Member

hugovk Jun 9, 2025

Suggested change

      
                days: 'Wed – Friday',
          
                days: 'Wednesday – Friday',

src/components/sections/theweek.astro

+                },
+                {
+                  id: 'saturday-sunday',
+                  days: 'Sat – Sun',

Member

hugovk Jun 9, 2025

Suggested change

      
                days: 'Sat – Sun',
          
                days: 'Saturday – Sunday',

clytaemnestra reviewed

View reviewed changes

src/components/sections/theweek.astro

+                  id: 'saturday-sunday',
+                  days: 'Sat – Sun',
+                  dates: '19 – 20 July',
+                  activities: 'Sprints',

Contributor

clytaemnestra Jun 9, 2025

Suggested change

      
                activities: 'Sprints',
          
                activities: 'Sprints & Beginners' Day',

clytaemnestra reviewed

View reviewed changes

src/components/sections/theweek.astro

+                  id: 'monday-tuesday',
+                  days: 'Mon – Tues',
+                  dates: '14 – 15 July',
+                  activities: 'Workshops & Tutorials',

Contributor

clytaemnestra Jun 9, 2025

Suggested change

      
                activities: 'Workshops & Tutorials',
          
                activities: 'Tutorials',

clytaemnestra reviewed

View reviewed changes

src/components/sections/theweek.astro

+                <Headline id="theweek" title="The Week" center="true" />
+                <div class="grid sm:flex w-full justify-center sm:justify-around ">
+                  {weekPeriods.map((period) => (
+                  <div class="grid grid-cols-1 text-center w-[160px] m-5 rounded overflow-hidden">

Contributor

clytaemnestra Jun 9, 2025

I think it would look better if the text would be vertically centered. wdyt?

Contributor

artcz commented Jun 11, 2025

What about reusing the updates section for this? Since right now we don't have any call to actions left before the conference.

(text is a bit random just to illustrate the idea) – also pictures would need to be adjusted too :)

Contributor Author

nikoshell commented Jun 11, 2025

What about reusing the updates section for this? Since right now we don't have any call to actions left before the conference.

(text is a bit random just to illustrate the idea) – also pictures would need to be adjusted too :)

I create it in this PR
#1311

nikoshell changed the title ~~Add section "The Week"~~ Add section "The Week" mini version

Contributor

clytaemnestra commented Jun 13, 2025

already implemented in #1311

clytaemnestra closed this

egeakman deleted the ep2025-the-week branch

July 2, 2025 00:59

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

Labels

None yet