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

feat(folder): folder dropdown with all options #1851

Merged
merged 3 commits into from
Mar 31, 2025
Merged

Conversation

emilielr
Copy link
Collaborator

@emilielr emilielr commented Mar 26, 2025

Dropdown for å velge mappe


Motivasjon

Gjøre det enklere for brukere å flytte og velge en mappe ved opprettelse av tavle. Nå er det ikke mulig å ikke velge en mappe.

Endringer

  • Fjernet sjekkboksen for om det skal være en privat tavle -> det er erstattet med et valg i dropdownen som heter "Ingen mappe"
  • Når man står i en spesifikk mappe, så er dette default valgt når man oppretter ny
  • Når man står ved oversikten, så er default "Ingen mappe" valgt
Før Etter
image image

Sjekkliste for Review

  • Når man oppretter en tavle i en spesifikk mappe, så er denne satt som default
  • Når man oppretter en tavle i oversikten, så er denne satt som "Ingen mappe"
  • Uansett hvor man flytter en tavle i en mappe (på rediger-siden), så slettes tavleIDen fra enten bruker eller organisasjon i databasen (avhengig om den lå i en organisasjon eller på bruker tidligere), og tavleIden blir lagt til på den nye plassen (enten ny organisasjon eller på bruker (hvis Ingen mappe er satt)).

Copy link
Contributor

@astoervold astoervold left a comment

Choose a reason for hiding this comment

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

Noen småkommentarer, de fleste for egen læring 🕺

@emilielr emilielr merged commit 4152394 into main Mar 31, 2025
3 checks passed
@emilielr emilielr deleted the set-specific-folder branch March 31, 2025 08:52
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