Skip to content

Conversation

@sapnilaa
Copy link
Contributor

  • 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.

@sapnilaa sapnilaa requested a review from a team as a code owner January 15, 2026 13:48
@sapnilaa
Copy link
Contributor Author

sapnilaa commented Jan 15, 2026

Resolves #112

Copy link
Member

@orjahren orjahren left a 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 gjør den en redeployment? Bare en tanke, vær gjerne uenig i den 😅

@sapnilaa
Copy link
Contributor Author

Syns selv å redeploye hver dag er litt ganske dumt.

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 gjør den en redeployment?

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 if nøkkelordet for å kjøre noe dersom det er lagt til et nytt event for FUI på Peoply og tenkte kanskje å bruke det, med mindre det er en lettere måte å ordne workflow-en på? 🤔

@orjahren
Copy link
Member

Syns selv å redeploye hver dag er litt ganske dumt.

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 gjør den en redeployment?

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 if nøkkelordet for å kjøre noe dersom det er lagt til et nytt event for FUI på Peoply og tenkte kanskje å bruke det, med mindre det er en lettere måte å ordne workflow-en på? 🤔

Foreslår at vi tar dette på Slack :) https://fui-ifi.slack.com/archives/C0KM8D4QH/p1769104584131999

return {...event, startDate: updatedStartDate };
})

fs.writeFileSync("_data/events.json", JSON.stringify(updatedData, 2));
Copy link
Member

Choose a reason for hiding this comment

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

Burde gjenbruke variabelen som holder filnavnet ;)

Image

Copy link
Member

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??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Uff, takk haha!

Copy link
Contributor Author

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.

Copy link
Contributor Author

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? 😅


- name: Trigger main build workflow
if: steps.check.conclusion == 'failure'
uses: benc-uk/workflow-dispatch@v1
Copy link
Member

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?

Copy link
Contributor Author

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

Copy link
Contributor Author

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 ^

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