Skip to content

Fix event time to follow correct timezone - #118

Merged
sapnilaa merged 6 commits into
masterfrom
fix-event-time
Oct 31, 2025
Merged

sapnilaa merged 6 commits into
masterfrom
fix-event-time

Conversation

@sapnilaa

Copy link
Copy Markdown
Contributor

La til et Python-script som forandrer på startDate-variabelen til å følge tidssone i Oslo. La til to linjer i ci.yml-filen som kjører scriptet etter å ha hentet events fra Peoply.

Tror også at det skal kunne oppdatere seg basert på om det er sommertid eller vintertid 🙏.

@orjahren

Copy link
Copy Markdown
Contributor

Kult! Hva med å evt gjøre dette i JS der koden kjøres?

@sapnilaa sapnilaa changed the title Fix event time to follow correct timezone #117 Fix event time to follow correct timezone Oct 31, 2025
@sapnilaa

Copy link
Copy Markdown
Contributor Author

Slik at man slipper at scriptet er en del av CI?

@sapnilaa

Copy link
Copy Markdown
Contributor Author

Nå gjøres dette direkte inn i fetch-events.js.

@sapnilaa
sapnilaa marked this pull request as ready for review October 31, 2025 13:33
@sapnilaa
sapnilaa requested a review from a team as a code owner October 31, 2025 13:33
@sapnilaa

Copy link
Copy Markdown
Contributor Author

Resolve #117

@orjahren orjahren left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, bare å merge hvis du har testet at det funker (ref localeString)

Comment thread .github/fetch-events.js Outdated
Comment thread .github/fetch-events.js Outdated
Comment thread .github/fetch-events.js
@sapnilaa
sapnilaa merged commit 4451b0e into master Oct 31, 2025
5 checks passed
@orjahren
orjahren deleted the fix-event-time branch October 31, 2025 15:21

This branch was previously deployed

1 inactive deployment
github-pages — 4451b0e2 Deployed Oct 31, 2025 by sapnilaa via deploy #17
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.

2 participants