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

ChooseRightsPage #503

Closed
Albertlarsen opened this issue Aug 23, 2023 · 17 comments
Closed

ChooseRightsPage #503

Albertlarsen opened this issue Aug 23, 2023 · 17 comments
Assignees
Labels
kind/feature-request Used when issue is a new feature or request

Comments

@Albertlarsen
Copy link
Contributor

Description

Create a new page that allows users to select and manage permissions for different applications. This page will provide users with the ability to customize the access rights they want to grant to a person for an application.

Implement the page in the picture below:

image

Additional Information

No response

@Albertlarsen Albertlarsen added the kind/feature-request Used when issue is a new feature or request label Aug 23, 2023
@Albertlarsen Albertlarsen self-assigned this Aug 23, 2023
@Randiabs
Copy link

Hvor kan vi teste akkurat dette bildet ? @Albertlarsen på de testpersonene æ har vært inne på får æ ikke opp de som man ikke har all rettighet på.

@Albertlarsen
Copy link
Contributor Author

@Randiabs bare velg alle servicene på side 1 og åpne en av de gule

@Albertlarsen
Copy link
Contributor Author

Kan du legge ved screenshot av hvor det ikke går?

@Randiabs
Copy link

Litt problemer med å komme inn på test akkurat nå, men har bare fått grønne når æ ha testa tidligere. Kan sjekke senere. Evt kanskje du har en lenke til dette ? Velge alle servicene, altså hva mener du da ?

@Albertlarsen
Copy link
Contributor Author

Mener velge alle tjenestene og legge de i "handlekurven" på startsiden for delegering av enkeltrettigheter

@Randiabs
Copy link

Randiabs commented Sep 15, 2023

Da kom æ mæ inn på det:
Noen grunn til at det innholdet som kommer opp i gul ramme, ikke stemmer overens med bildet fra skissene her opp ???
Altså typ oppsett av innholdet, hvorfor ligger rettigheter du ikke får delegert inne i alert....?
Se bildet fra test:

Image

@Albertlarsen
Copy link
Contributor Author

Albertlarsen commented Sep 15, 2023

Syns det var litt smør på flesk at det stod først "Nedenfor ser du hvilke rettigheter dette gjelder" og "Disse rettighetene kan du ikke delegere" og da fjernet jeg "Nedenfor ser du hvilke rettigheter dette gjelder". Da ga det ikke lenger mening å ha de chipene for seg selv.

Også har vi ikke det ikonet eller komponenten på bildet under. Men vi kan sikkert snakke om det på design-møtet på mandag så finner vi ut av hva vi kan gjøre der 😊
image

@Albertlarsen
Copy link
Contributor Author

Kan dere kommentere med oppdaterte bilder på issuen for hvordan dere vil ha det når disse er ferdig @Randiabs @hannekristin? 😊

@Albertlarsen
Copy link
Contributor Author

Albertlarsen commented Sep 28, 2023

Note to self: Fiks til å bli slik og legg til litt mer margin mellom nest nederste og nederste tekst i alerten.

Image

@hannekristin
Copy link
Contributor

@Albertlarsen Oppdaget tilfeldigvis at teksten i tjenesten ikke er slik som vi har laget den. Denne setningen her kan misforstås og må fjernes:

Image

@Albertlarsen
Copy link
Contributor Author

Albertlarsen commented Nov 29, 2023

Allright, legger inn en fix på det så kommer fjerningen av teksten inn i miljøene snart @hannekristin

@allinox
Copy link
Contributor

allinox commented Dec 13, 2023

@Albertlarsen @hannekristin
Jeg prøver å rydde litt i backloggen vår - hva er status på denne tasken? 😊

@Albertlarsen
Copy link
Contributor Author

@allinox Den er fikset ja, har du fått ansvar over å rydde i design-review kolonna?

@allinox
Copy link
Contributor

allinox commented Dec 13, 2023

@Albertlarsen
Nei, jeg bare går igjennom issuene på frontend-repo og prøver å lukke de som kan/bør lukkes (som de som er blitt obviated av andre tasker eller har ligget urørt en stund, slik som denne). Så blir det litt lettere for oss å holde oversikt 😊

Men om denne da er done og Hanne ikke har noen innspill ila dagen så lukker jeg den 👍

@hannekristin
Copy link
Contributor

@Albertlarsen @allinox Det er også feil i overskriften på denne siden. "Her kan du velge hvilke rettigheter du vil delegere til ANNEMA FIGMA" må endres til: Du gir disse rettighetene til ANNEMA FIGMA
image

@allinox
Copy link
Contributor

allinox commented Dec 13, 2023

@hannekristin
Den er grei - jeg kan fikse det så vi får det unnagjort 😊
(Siden jeg er between tasks uansett)

@hannekristin
Copy link
Contributor

Testet ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature-request Used when issue is a new feature or request
Projects
Development

No branches or pull requests

4 participants