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

Etiketter på planneroppgaver #1084

Merged
merged 5 commits into from
Apr 18, 2023
Merged

Etiketter på planneroppgaver #1084

merged 5 commits into from
Apr 18, 2023

Conversation

Remi749
Copy link
Collaborator

@Remi749 Remi749 commented Apr 18, 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

Det er nå mulig å angi etiketter på planneroppgaver. Dette gjøres ved å legge inn etiketter, separert med semikolon i den nye kolonnen på Planneroppgaver listen(e). Under provisjonering vil den legge sammen alle etikettene som er angitt på listen, opprette disse i planneren og angi etikettene for hvert element i Planneroppgaver webdelen.

image

Dersom det ikke angis noen etiketter vil det heller ikke komme med etiketter. Vi følger begrensningen på 25 etiketter, dersom du angir flere enn 25 etiketter vil de overskytende ikke følge med under provisjonering

Etiketter er stresstestet og det er gjort testing med tanke på tidsbruk:

  • Uten etiketter:
    59 elementer, ca 40 sekunder

  • Med etikketer på hvert eneste element, alt fra 2-6 etiketter per element
    59 elementer, ca 1min 45 sekunder

Hvordan teste

Vennligst beskriv hvordan noen andre (en vanlig bruker uten kodeferdigheter) kan teste denne PR-en. Følg eksempelet under, punktene du legger ved vil bli brukt når vi tester neste versjon av Prosjektportalen

# Handling Forventet resultat
1 Angi etiketter på planneroppgaver ved å legge inn etiketter, separert med semikolon i den nye kolonnen på Planneroppgaver listen(e). Etikettene legges til i planneren og vises på hvert element i Planneroppgaver webdelen.
2 Ikke angi noen etiketter. Ingen etiketter vises i planneren.
3 Angi mer enn 25 etiketter. Kun de første 25 etikettene følger med under provisjonering, og de overskytende blir ikke opprettet i planneren.

Relevante issues (hvis aktuelt)

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

@Remi749 Remi749 added complexity: medium Issue/PR with medium complexity backend Related to backend/apis labels Apr 18, 2023
@Remi749 Remi749 added this to the 1.9 milestone Apr 18, 2023
@Remi749 Remi749 self-assigned this Apr 18, 2023
Copy link
Collaborator

@olemp olemp left a comment

Choose a reason for hiding this comment

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

Lagt inn en kommentar. Er noen av filene som er endret ikke en del av planneroppgaver-endringen @Remi749?

Copy link
Collaborator

@olemp olemp left a comment

Choose a reason for hiding this comment

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

🦩

@Remi749 Remi749 merged commit 11c2761 into releases/1.9 Apr 18, 2023
@Remi749 Remi749 deleted the issues/1083 branch April 18, 2023 10:39
Remi749 added a commit that referenced this pull request Apr 20, 2023
@olemp olemp mentioned this pull request May 12, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend/apis complexity: medium Issue/PR with medium complexity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants