-
Notifications
You must be signed in to change notification settings - Fork 29
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
Nye felt for Aktiviteter : Fra hvilket tidspunkt kan aktiviteten være synlig ? #1056
The head ref may contain hidden characters: "1055-feature-mulighed-for-at-skjule-aktiviteter-fra-for\u00E6ldrene"
Nye felt for Aktiviteter : Fra hvilket tidspunkt kan aktiviteten være synlig ? #1056
Conversation
Default er timezone.now(), så nye aktiviteter per default er synlige når de er lavet (og såfremt de andre værdier er OK, f.x. at aktiviteten ender i fremtiden etc.).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ser korrekt ud. Og checkede at vi generelt bruger timezone.now
til at checke datoer
Før vi merger denne bør vi drøfte om vi skal have en mulighed for at aktiviteten bare ikke skal vises. Med dette går vi ud fra at aktiviteten på et eller andet tidspunkt skal vises. For de fleste aktiviteter er det tilfældet, men der kan være aktiviteter vi opretter, og senere finder ud af, vi ikke ønsker at vise. Det kunne f.eks. være Game Jam når der er udsolgt, der kan det være vi gerne vil skjule aktiviteten og i stedet vise en ventelisteaktivitet. Jeg forstår godt ideen med at det er en dato, og i de fleste tilfælde er jeg enig, men i dette tilfælde er jeg ikke sikker på, om jeg er enig. Derfor ved jeg ikke om jeg er enig i at det løser #1055. |
Kan det løses ved at man blot sætter dato til år 9999? 🙂 Jeg synes en "publiserings-dato" er en god feature, som sikrer at man ikke glemmer at gøre aktiviteten synlig senere. |
Jeg er enig i, at det er en god feature, og det er også derfor jeg er lidt splittet. Hvilket jeg også har skrevet til Mikael tidligere. Man kan godt sætte årstallet langt ud i fremtiden, men det er ikke intuitivt - og det er jo noget af det vi forsøger at gøre systemet. Mit forslag ville være at vi havde begge ting. En dato, hvor den bliver udgivet. Og en tjekboks der styrer om den må vises. Hvis tjekboksen er sat til den må vises, og datoen er i dag eller senere vises aktiviteten. Ellers vises den ikke. |
PR Summary
|
Default er timezone.now(), så nye aktiviteter per default er synlige når de er lavet (og såfremt de andre værdier er OK, f.x. at aktiviteten ender i fremtiden etc.).