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

313 let admin choose when applicant info will be hidden #331

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

sindremil
Copy link
Collaborator

@sindremil sindremil commented Oct 25, 2024

Ready for review

@sindremil sindremil linked an issue Oct 25, 2024 that may be closed by this pull request
Copy link

vercel bot commented Oct 25, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
online-opptak ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 27, 2024 1:03pm

@julian-ao
Copy link
Member

Kan du legge til bilde av hvordan det ser ut?

@sindremil
Copy link
Collaborator Author

Når man oppretter ny periode (info er bare synlig onhover på infoikonet):
image

Når søkerne er skjulte:
image

Når søkerne ikke er skjulte:
image

@jorgengaldal
Copy link
Collaborator

Hadde kanskje vært lurt å legge inn valg for antall dager det er skjult?

@julian-ao
Copy link
Member

julian-ao commented Oct 26, 2024

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

@sindremil
Copy link
Collaborator Author

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

@jorgengaldal
Copy link
Collaborator

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

@sindremil
Copy link
Collaborator Author

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

@sindremil
Copy link
Collaborator Author

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

Men isåfall så var dette feil ved opptaket som var. Meg som ikke var admin kunne ikke se info om søkerne når det var opptak.

@jorgengaldal
Copy link
Collaborator

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

Men isåfall så var dette feil ved opptaket som var. Meg som ikke var admin kunne ikke se info om søkerne når det var opptak.

Ikke etter søknadsfristen var gått ut engang?

@sindremil
Copy link
Collaborator Author

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

Men isåfall så var dette feil ved opptaket som var. Meg som ikke var admin kunne ikke se info om søkerne når det var opptak.

Ikke etter søknadsfristen var gått ut engang?

Jo tror man kunne se søkerne etter fristen gikk ut.

sindremil and others added 2 commits October 27, 2024 13:26
Co-authored-by: Julian Ammouche Ottosen <[email protected]>
Co-authored-by: Julian Ammouche Ottosen <[email protected]>
@jorgengaldal
Copy link
Collaborator

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

Men isåfall så var dette feil ved opptaket som var. Meg som ikke var admin kunne ikke se info om søkerne når det var opptak.

Ikke etter søknadsfristen var gått ut engang?

Jo tror man kunne se søkerne etter fristen gikk ut.

Ja, men da tror jeg det har vært riktig. Er jo også sånn det står i koden nå.

@sindremil
Copy link
Collaborator Author

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

Men isåfall så var dette feil ved opptaket som var. Meg som ikke var admin kunne ikke se info om søkerne når det var opptak.

Ikke etter søknadsfristen var gått ut engang?

Jo tror man kunne se søkerne etter fristen gikk ut.

Ja, men da tror jeg det har vært riktig. Er jo også sånn det står i koden nå.

Åh jeg burde lest mer nøye, men ser hvordan det funker nå. Søkerne er skjulte frem til søkefristen er over, og så blir de skjulte igjen 7 dager etter intervjuperioden er over. Skal endre info popupen så det gir mer mening.

@sindremil
Copy link
Collaborator Author

Jeg skjønner ikke helt funksjonaliteten, sikkert jeg som misforstår. Hvorfor vil vi skjule denne infoen, burde ikke komiteene få vite tlf nummer osv? Og hvorfor skal ikke infoen forbli skjult etter 7 dager etter intervjuperioden?

Hvorfor infoen er skjult og hvorfor det opphører 7 dager etter intervjuperioden vet jeg ikke, men det var hardkodet inn. Alt jeg har gjort er å la admin slå dette av når de oppretter en periode.

Skal vel være motsatt? At infoen skjules etter 7 dager? Tanken er vel at ikke de nye komitémedlemmene skal få se søknadene til de andre og hvem som har søkt.

Ah det kan stemme. Har uansett ikke rørt funksjonaliteten som styrer når søkerne går mellom skjult og ikke skjult. Har bare lagt på en ekstra condition som er feltet jeg adda til periode typen. Men kan endre på teksten i ingo popupen så det gir mer mening.

Men isåfall så var dette feil ved opptaket som var. Meg som ikke var admin kunne ikke se info om søkerne når det var opptak.

Ikke etter søknadsfristen var gått ut engang?

Jo tror man kunne se søkerne etter fristen gikk ut.

Ja, men da tror jeg det har vært riktig. Er jo også sånn det står i koden nå.

Åh jeg burde lest mer nøye, men ser hvordan det funker nå. Søkerne er skjulte frem til søkefristen er over, og så blir de skjulte igjen 7 dager etter intervjuperioden er over. Skal endre info popupen så det gir mer mening.

Tenker søkere burde bli skjult 7 dager etter intervjuperioden uansett. Har nå gjort slik at man bare bestemmer om søkerne skal være skjulte før søkeperioden er over.

@fredrir
Copy link
Member

fredrir commented Oct 27, 2024

Hadde ikke det vært en bedre ide, å la admin skjule info om søkere inne på /admin/[period-id], når opptaket er live, istedenfor når opptaket blir laget. Eventuelt at admin kan velge datoen infoen blir skjult?

Husker at bakgrunnen for at valget var på 7 dager etter intervjuperioden, var at komiteene skulle ha info om søkerne helt fram til fordelingsmøtet + 1 dag, slik at man hadde telefonnummeret til de man skulle ringe. Men at det var viktig at de ny opptatte ikke skulle ha tilgang på søkerne. Altså burde vi fjerne at 7 dager er hard-kodet inn, og heller ha en dynamisk approach. Da det ikke vil passe alle opptak.

Så tenker en ide, er at man enten lar admin velge gjennom en date input når søker info skal skjules, eller ha en knapp på dashbordet som skjuler søkerne

@sindremil
Copy link
Collaborator Author

Hadde ikke det vært en bedre ide, å la admin skjule info om søkere inne på /admin/[period-id], når opptaket er live, istedenfor når opptaket blir laget. Eventuelt at admin kan velge datoen infoen blir skjult?

Husker at bakgrunnen for at valget var på 7 dager etter intervjuperioden, var at komiteene skulle ha info om søkerne helt fram til fordelingsmøtet + 1 dag, slik at man hadde telefonnummeret til de man skulle ringe. Men at det var viktig at de ny opptatte ikke skulle ha tilgang på søkerne. Altså burde vi fjerne at 7 dager er hard-kodet inn, og heller ha en dynamisk approach. Da det ikke vil passe alle opptak.

Så tenker en ide, er at man enten lar admin velge gjennom en date input når søker info skal skjules, eller ha en knapp på dashbordet som skjuler søkerne

God ide! Kan legge inn at man heller velger datoen søkerne blir skjult. Angående det å la admin skjule info om søkere inne på /admin/[period-id], vil jeg si det går mer under #330 .

@sindremil
Copy link
Collaborator Author

Hadde ikke det vært en bedre ide, å la admin skjule info om søkere inne på /admin/[period-id], når opptaket er live, istedenfor når opptaket blir laget. Eventuelt at admin kan velge datoen infoen blir skjult?

Husker at bakgrunnen for at valget var på 7 dager etter intervjuperioden, var at komiteene skulle ha info om søkerne helt fram til fordelingsmøtet + 1 dag, slik at man hadde telefonnummeret til de man skulle ringe. Men at det var viktig at de ny opptatte ikke skulle ha tilgang på søkerne. Altså burde vi fjerne at 7 dager er hard-kodet inn, og heller ha en dynamisk approach. Da det ikke vil passe alle opptak.

Så tenker en ide, er at man enten lar admin velge gjennom en date input når søker info skal skjules, eller ha en knapp på dashbordet som skjuler søkerne

Forresten, husker du hvorfor søkerne holdes skjult under søknadsperioden?

@fredrir
Copy link
Member

fredrir commented Oct 30, 2024

Hadde ikke det vært en bedre ide, å la admin skjule info om søkere inne på /admin/[period-id], når opptaket er live, istedenfor når opptaket blir laget. Eventuelt at admin kan velge datoen infoen blir skjult?
Husker at bakgrunnen for at valget var på 7 dager etter intervjuperioden, var at komiteene skulle ha info om søkerne helt fram til fordelingsmøtet + 1 dag, slik at man hadde telefonnummeret til de man skulle ringe. Men at det var viktig at de ny opptatte ikke skulle ha tilgang på søkerne. Altså burde vi fjerne at 7 dager er hard-kodet inn, og heller ha en dynamisk approach. Da det ikke vil passe alle opptak.
Så tenker en ide, er at man enten lar admin velge gjennom en date input når søker info skal skjules, eller ha en knapp på dashbordet som skjuler søkerne

Forresten, husker du hvorfor søkerne holdes skjult under søknadsperioden?

Årsaken var mest for at folk i komiteene ikke skulle se at hvis det var noen de kjente som f.eks. et fadderbarn ikke hadde søkt, at de skulle oppfordre dem til å søke. Eller at man kunne se hvis noen som hadde søkt trakk søknaden.

@julian-ao
Copy link
Member

Hva er status på dette @sindremil @fredrir @jorgengaldal?

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.

Let admin choose when applicant info will be hidden
4 participants