Skip to content

Conversation

@atlelill
Copy link
Contributor

Dette er en fil som leter etter issues med prefixet "nve-" og kopierer og oppdaterer de over til et Project som da er i sync med issues.

Dette er en fil som leter etter issues med prefixet "nve-" og kopierer og oppdaterer de over til et Project som da er i sync med issues.
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-713.westeurope.5.azurestaticapps.net

@malingranlynve malingranlynve changed the title Create deisgnsystem-project-sync.yml feat(dokumentasjon): Create designsystem-project-sync.yml Nov 26, 2025
@malingranlynve
Copy link
Contributor

Lisa og jeg har tatt en gjennomgang og synes det ser greit ut, vi prøver! 🤠 Vi har bare et små kommentarer angående variabelnavn og struktur. Vi trenger mer forklarende navn slik at vi senere klarer å lese hva som skjer i koden. Ett eksempel var const m =, dette sier jo oss ikke så mye og vi må tolke mye mer! Vil du at vi skal gå inn i branchen å fikse det vi tenker? Eller skal vi legge igjen kommentarer her?

@malingranlynve malingranlynve changed the title feat(dokumentasjon): Create designsystem-project-sync.yml feat(dokumentasjon): Implementere designsystem-project-sync.yml Nov 26, 2025
Changed variablename m to externalUrl
@atlelill
Copy link
Contributor Author

atlelill commented Dec 3, 2025

@malingranlynve Kan du sjekke fila på nytt? Det eneste jeg gjorde i den siste commit'en var å endre variabelnavnet m til externalUrl. Er det bare for meg å klikke på "Squash and merge" for å kunne sjekke om dette funker?

Copy link
Contributor

@lisamarimyreneNVE lisamarimyreneNVE left a comment

Choose a reason for hiding this comment

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

Det ser bra ut. Vi kan ta det inn og teste om det funker, så kan vi heller endre på ting senere :)

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-713.westeurope.5.azurestaticapps.net

@atlelill atlelill merged commit f3938ed into main Dec 3, 2025
7 of 8 checks passed
@atlelill atlelill deleted the atlelill-patch-2 branch December 3, 2025 12:09
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.

4 participants