Skip to content
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

Fiks for feilmelding i Prosjektinformasjon #1091

Merged
merged 2 commits into from
Apr 19, 2023
Merged

Fiks for feilmelding i Prosjektinformasjon #1091

merged 2 commits into from
Apr 19, 2023

Conversation

olemp
Copy link
Collaborator

@olemp olemp commented Apr 19, 2023

Pull request (PR)

Sørg for at du ber om PR for din branch (høyre side). Sørg for at du gjør en PR mot riktig release-branch (venstre side). Sjekk commits og alle commit-meldingene.

Sjekklisten din

Alle sjekkpunktene under må være sjekket av og godkjent for at vi skal kunne merge branchen din mot dev.

  • Sjekk at din branch ikke feiler på linting.
  • Legg ved beskrivelse i CHANGELOG, markert med ID av issue knyttet til PR-en
  • Angi korrekt Milestone på PR-en og issuet, samt tilegn deg selv PR-en og legg til labels

Beskrivelse

Fiks for feilmelding i Prosjektinformasjon fra #1086. Vi henter nå kun ut èn statusrapport (hentet tidligere ut alle publiserte selv om vi bare brukte siste publiserte).

Relevante issues (hvis aktuelt)

#1086

Sjekkliste for godkjenner

Alle sjekkpunktene under må være sjekket av og godkjent av reviewers for at vi skal kunne merge branchen din mot dev.

  • Sjekk at det er fylt ut testpunkter
  • Sjekk om det er nødvendig å nevne denne PR i release notes
  • Sjekk om det er nødvendig å oppdatere dokumentasjon for hjelpeinnhold

@olemp olemp linked an issue Apr 19, 2023 that may be closed by this pull request
@olemp olemp added this to the 1.8.2 milestone Apr 19, 2023
@olemp olemp added the bug Something isn't working label Apr 19, 2023
@olemp olemp self-assigned this Apr 19, 2023
@olemp olemp changed the base branch from main to releases/1.8 April 19, 2023 11:55
@olemp olemp marked this pull request as ready for review April 19, 2023 12:03
@olemp olemp requested review from tarjeieo, okms and Remi749 as code owners April 19, 2023 12:03
@olemp olemp merged commit b8a042d into releases/1.8 Apr 19, 2023
@olemp olemp deleted the issues/1086 branch April 19, 2023 12:22
@olemp olemp mentioned this pull request May 12, 2023
6 tasks
@pzljanb
Copy link
Contributor

pzljanb commented May 24, 2023

Hvordan teste denne @tarjeieo ?

@tarjeieo
Copy link
Member

Du kan egentlig bare teste at prosjektstatus laster på forsida av prosjekter. Før så feilet den dersom det var mange tidligere og fyldige prosjektstatuser, da vi henta ut alle - nå henter vi bare siste. Men ikke helt enkelt å være 100% sikker på at man har fått testa den riktig nei.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feilmelding i prosjektinformasjon
3 participants