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 kommandolinje og standard egenskaper på prosjekttidslinje #1043

Merged
merged 5 commits into from
Mar 8, 2023

Conversation

Remi749
Copy link
Collaborator

@Remi749 Remi749 commented Mar 7, 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 dev-branchen (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
  • Tilegn deg selv PR-en og legg til labels

Beskrivelse

Egenskaper på prosjekttidslinje ble endret fra 1.7.2 til 1.8.0, disse var satt i maler for prosjekt og program. Dette førte til at kommandolinje ikke ble vist vist som standard (for eksisterende prosjekter blandt annet)

showCmdTimelineList blir satt til true, men siden denne ikke finnes (som nå heter showTimelineListCommands) blir den false.

Dette er blitt fikset, riktig properties er angitt i maler og upgrade script. Har også fjerna showFilterButton property som ikke var i bruk.

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 Gå til en Prosjektidslinje og verifiser at kommandolinje dukker opp Kommandolinje skal dukke opp på Prosjekttidslinje

Relevante issues (hvis aktuelt)

@Remi749 Remi749 added bug Something isn't working complexity: small Issue/PR with small complexity labels Mar 7, 2023
@Remi749 Remi749 added this to the 1.8.1 milestone Mar 7, 2023
@Remi749 Remi749 requested review from olemp, tarjeieo and okms as code owners March 7, 2023 13:58
@Remi749 Remi749 self-assigned this Mar 7, 2023
@tarjeieo tarjeieo merged commit 3084019 into releases/1.9 Mar 8, 2023
@tarjeieo tarjeieo deleted the issues/1042 branch March 8, 2023 08:18
Remi749 added a commit that referenced this pull request Mar 10, 2023
…1043)

* Update WebPart props for ProjectTimeline (UpgradeAllSites...)

* Update _JsonTemplates to set correct props for Timeline webpart

* Remove showFilterButton boolean - no longer in use

* Add ProjectTimeline default props

* Changelog
olemp pushed a commit that referenced this pull request Mar 24, 2023
…1043)

* Update WebPart props for ProjectTimeline (UpgradeAllSites...)

* Update _JsonTemplates to set correct props for Timeline webpart

* Remove showFilterButton boolean - no longer in use

* Add ProjectTimeline default props

* Changelog
olemp pushed a commit that referenced this pull request Mar 29, 2023
…1043)

* Update WebPart props for ProjectTimeline (UpgradeAllSites...)

* Update _JsonTemplates to set correct props for Timeline webpart

* Remove showFilterButton boolean - no longer in use

* Add ProjectTimeline default props

* Changelog
olemp pushed a commit that referenced this pull request Mar 29, 2023
…1043)

* Update WebPart props for ProjectTimeline (UpgradeAllSites...)

* Update _JsonTemplates to set correct props for Timeline webpart

* Remove showFilterButton boolean - no longer in use

* Add ProjectTimeline default props

* Changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working complexity: small Issue/PR with small complexity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants