-
Notifications
You must be signed in to change notification settings - Fork 6
Add schedule to redeploy every nigth at 02:00 #123
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
base: master
Are you sure you want to change the base?
Conversation
sapnilaa
commented
Jan 15, 2026
- Redeployer kl 02:00 hver natt, godt mulig at dette er altfor ofte 😅.
- Ser ikke ut som at noe må endres med fargen til margen.
|
Resolves #112 |
orjahren
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kult! Vet ikke helt hva jeg syns her. Vet vi noe om hvor mye en redeployment koster oss? Kan vel umulig være veldig mye. Men hver natt er likevel kanskje likevel litt ofte mtp hvor sjeldent det vil kunne tenkes å være endringer 🤔
Klarer vi å tenke på noe mer bigbrain? Hva med å kjøre en "billig" jobb hver natt som kun sjekker om det har kommet nye arrangementer på Peoply sammenlignet med det som ligger på nettsiden og hvis det har det så gjør den en redeployment? Bare en tanke, vær gjerne uenig i den 😅
|
Syns selv å redeploye hver dag er litt ganske dumt.
Ja syns det var mye lurere! Skal dennes sjekken gjøres før byggingen? eller etter bygging og før deployment? Så at man kunne bruke |
Foreslår at vi tar dette på Slack :) https://fui-ifi.slack.com/archives/C0KM8D4QH/p1769104584131999 |
.github/fetch-events.js
Outdated
| return {...event, startDate: updatedStartDate }; | ||
| }) | ||
|
|
||
| fs.writeFileSync("_data/events.json", JSON.stringify(updatedData, 2)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hvis dette er ment å være samme filen, that is. Hvorfor er det 2??
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uff, takk haha!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eller jo, det er 2, en for kun event-id og en for selve json-data fra Peoply.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Er det dumt å ha to filer? 😅
.github/workflows/check-update.yml
Outdated
|
|
||
| - name: Trigger main build workflow | ||
| if: steps.check.conclusion == 'failure' | ||
| uses: benc-uk/workflow-dispatch@v1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hva er dette og hva gjør den? Er det nødvendig å bruke en ekstern action?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Den starter hovedworkflow-en når det kommer en ny event i Peoply fra det jeg forstod
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fant ikke en annen måte å gjøre det på enn ^
