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

Småfikser for delte tavler #539

Merged
merged 2 commits into from
Nov 24, 2021
Merged

Småfikser for delte tavler #539

merged 2 commits into from
Nov 24, 2021

Conversation

magnusrand
Copy link
Contributor

Etter at den nye delte-tavler-funksjonaliteten ble lagt til i staging ble noen områder for forbedring funnet. Denne PRen inneholder de relevante fiksene.

Inkluderer:

  • Varsel med modal om at du må være owner er nå basser på listen av owner fra settingsContext og ikke snapshotet som hentes ved update. Dette er gjort av hensyn til hastighet. settingsContext er tilgjengelig for oss før ownersData er, noe som gjør at man unngår å i noen tilfeller få opp modalen når man faktisk er en eier, men venter på å fetche data. Dette burde heller ikke gå utover sikkerheten, da selv om man skulle greie å manipulere settings til å si at man er en owner, vil fortsatt ingen kall gjort til firestore om å endre ting fungere på grunn av sikkerhetsregler og -sjekker.
  • Når man skal fjerne en eier eller invitasjon fra en tavle får man nå opp et lite loading-ikon i IconButton-elementet i tillegg til at knappen blir satt til disabled. Dette gjør at i situasjoner hvor kallet går tregt så virker ikke tavla lenger uresponsiv.
  • Fanenavnet til der man mottar informasjon om delte tavler er endret fra «Delt med meg» til «Invitasjoner» da dette gir en bedre forståelse av hva fanen fakisk er.
  • Hvis man mister tilgang til en tavle mens man er inne på Rediger vil nå siden lastes inn på nytt og man blir møtt med en infoskjerm om at man ikke lenger har tilgang til tavlen.

Fixed problems discovered in staging. Includes:
- Modal warning about needing to be an owner is now based on settingsContext to be faster
- A loading icon is displayed when a request or owner is removed from a board
- "Delt med meg" is changed to "Invitasjoner"
@magnusrand magnusrand requested a review from a team November 24, 2021 12:59
@magnusrand magnusrand merged commit ae07e63 into master Nov 24, 2021
@magnusrand magnusrand deleted the fix/fixes-for-shared-boards branch November 24, 2021 15:18
@magnusrand magnusrand restored the fix/fixes-for-shared-boards branch November 25, 2021 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants