Skip to content

Eksponerer branchnavn og lager endepunkt for å hente det for frontend #3374

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

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

hanneolsen
Copy link
Contributor

@hanneolsen hanneolsen commented Oct 14, 2024

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-22519

💰 Hva forsøker du å løse i denne PR'en

Henter ut og viser branchnavn og sha1 for frontend og backend, slik at vi enkelt kan se hva som ligger ute i et miljø. Dette gjøres i alle miljøer by default, selv om oppgaven spesifikt gikk på preprod. Siden informasjonen er skjult by default burde ikke dette være et problem. Jeg har lagt opp til at man kan lage en enkel bookmarklet for å toggle visning av denne komponenten. URLen til dette ligger inni komponent-fila, men jeg legger den her også for ekstra dokumentasjon:

javascript: (() => {
var el=document.getElementById("vis-github-info");
if(el.checkVisibility()){el.style.setProperty('display','none');}
else{el.style.setProperty('display','block');}}
)();

🔎️ Er det noe spesielt du ønsker å fremheve?

Hvis noen har en bedre måte å sette env-variablene i github actions er det supert. Dette var den eneste måten jeg fant som fungerte.

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇

Jeg har ikke skrevet tester fordi:

🤷‍♀ ️Hvor er det lurt å starte?

Alt i ett

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

👀 Screen shots

Denne infoen ligger skjult by default:
image

Den kan sees ved å trigge en bookmarklet. Da ser den slik ut:
image

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.

1 participant