altinn-studio icon indicating copy to clipboard operation
altinn-studio copied to clipboard

12438 change name and design for tilgangsstyring tab in settings modal

Open JamalAlabdullah opened this issue 11 months ago • 5 comments

Description

  • Updated title text to be Oppstartskontroll instead of Tilgangsstyring Both in tab list and the actual title of the tab content.
  • Removed "Aktørkrav" * and added tabel to render checkboxes inside it.
  • Added logic to make sure that at least one checkbox should be choosen.
  • Added a modal to show warning message when user tries to uncheck the last checked checkbox.
  • Added helpText and docs link.
  • Updated and added some tests.

Related Issue(s)

  • #12438

Verification

  • [ ] Your code builds clean without any errors or warnings
  • [ ] Manual testing done (required)
  • [ ] Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

  • [ ] User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

JamalAlabdullah avatar Mar 14 '24 12:03 JamalAlabdullah

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 89.93%. Comparing base (b2168a7) to head (8bb075b). Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12511      +/-   ##
==========================================
+ Coverage   89.90%   89.93%   +0.02%     
==========================================
  Files        1287     1291       +4     
  Lines       18365    18395      +30     
  Branches     2236     2239       +3     
==========================================
+ Hits        16511    16543      +32     
+ Misses       1595     1593       -2     
  Partials      259      259              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Mar 14 '24 22:03 codecov[bot]

I think to fix the tests after we finish updating AccessControlTab component.

JamalAlabdullah avatar Mar 20 '24 12:03 JamalAlabdullah

Overskriftforslag @standeren og @Annikenkbrathen:

  1. Hvem kan starte denne appen?
  2. Hvem eier og kan starte denne appen?
  3. Velg hvem som skal eie og kan starte denne appen

I og med at vi har en beskrivende tekst under som forklarer det med "eie" så holder jeg en knapp på den enkleste varianten i nr. 1

Bruk av spørsmål i overskrifter må eventuelt harmoniseres med hva vi ellers gjør innenfor samme funksjonalitetsområde. Hvis vi ikke skal ha spørsmål, kan 1 også være "Velg hvem som kan starte appen"

Ildest avatar Mar 22 '24 11:03 Ildest

Overskriftforslag @standeren og @Annikenkbrathen:

  1. Hvem kan starte denne appen?
  2. Hvem eier og kan starte denne appen?
  3. Velg hvem som skal eie og kan starte denne appen

I og med at vi har en beskrivende tekst under som forklarer det med "eie" så holder jeg en knapp på den enkleste varianten i nr. 1

Bruk av spørsmål i overskrifter må eventuelt harmoniseres med hva vi ellers gjør innenfor samme funksjonalitetsområde. Hvis vi ikke skal ha spørsmål, kan 1 også være "Velg hvem som kan starte appen"

Jeg er enig i den enkleste varianten; og kanskje uten spm --> Velg hvem som kan starte appen 😄

standeren avatar Mar 26 '24 11:03 standeren

Something wierd with the css here making the left menu scrollable 🤔 https://github.com/Altinn/altinn-studio/assets/71079896/c693f1e7-80c1-4104-ac04-3bac37789f95

But it beginning to look good! 💯

Regarding the scroll, it looks like the problem should be fixed for the entir LeftNavigationstab, ? I have tested in dev and the scroll is also exists in tilgangsregler tab. so maybe this should be fixed in a seperate issue to fix all tabs toghether. ?

https://github.com/Altinn/altinn-studio/assets/90609090/c087ede4-d071-45ce-91a4-22917996b07e

JamalAlabdullah avatar Apr 02 '24 08:04 JamalAlabdullah