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

Feature/794: Opskrivninger til afdeling, angiv også # på venteliste #795

Conversation

mhewel
Copy link
Contributor

@mhewel mhewel commented Jan 2, 2023

image

@what-the-diff
Copy link

what-the-diff bot commented Jan 2, 2023

  • The template has been changed to show the place on the waiting list for a child.
  • A new variable is added in models/child_model: "waitinglists" which contains all departments and places that a child is signed up for, as tuples (department, place).
  • In views/membership_views we have made changes so that when you sign up or unsubscribe from a department's waitinglist it will update your position if there are other children who were already subscribed before you but now after signing up they should be placed behind you because of their age etc..

@codecov
Copy link

codecov bot commented Jan 2, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.17%. Comparing base (dc7274e) to head (e62d4c3).
Report is 527 commits behind head on master.

Current head e62d4c3 differs from pull request most recent head dd6fdb2

Please upload reports for the commit dd6fdb2 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #795      +/-   ##
==========================================
+ Coverage   62.58%   64.17%   +1.58%     
==========================================
  Files         140      133       -7     
  Lines        3654     3341     -313     
==========================================
- Hits         2287     2144     -143     
+ Misses       1367     1197     -170     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rasmus-unity
Copy link
Contributor

Hvor kommer ønsket om at vise venteliste placering fra?

Min fornemmelse er at flere afdelinger "plukker" fra ventelisten, f.eks. i forhold til lokal bopælskommune og piger først. Ved at vise placering, er der en risiko for at vi "lover" at nogle at de kommer med, hvis de har et lavt nummer på ventelisten.

Det giver i hvert fald ikke meget mening i Hedehusene, hvor vi er begyndt at starte med at sende invitationer ud til børn bosiddende i Høje Taastrup kommune. Risikerer det snarere vil skuffe forældre yderligere, når folk fra f.eks. Roskilde først bliver inviteret, hvis der stadig er ledige pladser efter de "lokale" har haft mulighed for at melde sig til.

@mhewel
Copy link
Contributor Author

mhewel commented Jan 2, 2023

venteliste nummeret bliver vist under "Se opskrivninger", f.x. som her:
image
Jeg har fået flere forespørgsler fra forældre her i afdelingen om hvor deres barn er på ventelisten, så jeg tænkte at det ikke var helt tydeligt

@mhewel
Copy link
Contributor Author

mhewel commented Jan 2, 2023

@rasmus-unity - interessant ting om vi skal gå væk fra at vise venteliste overhovedet.. Eller om der skal være noget mere info om hvad venteliste kan bruges til , skal der beskrives nogle generelle forbehold ? (f.x. invitation af tidligere deltagere først, så lokal kommune og evt andet ) ?

@lakridserne
Copy link
Contributor

Jeg tror begge pointer er valide. Mange afdelinger går slavisk efter nummer på ventelisten, men en del plukker også.

Mit forslag ville være at vi viser placeringen, men også tydeligt i toppen af hver venteliste skriver at tallet er vejledene, og afdelingerne kan, hvis de vurderer det er fordelagtigt for dem, afvige fra det. Det forpligter dog ikke afdelingen til at tage bestemte hensyn, det er alene afdelingen der bestemmer dette.

Den sidste del er vigtig fordi vi for eksempel i l Lyngby har et eksempel på en forældre, som forsøger at påvirke hvem vi tager ind.

@mhewel
Copy link
Contributor Author

mhewel commented Jan 2, 2023

Der er pt. flg beskrivelse på samme side ang ventelister - er der noget vi skal rette (måske nogle linjeskift) ?

Pladser på ventelister
Pladser på venteliste af vejledende. Den enkelte afdeling kan vælge at prioritere børn længere nede på listen for at få den bedst mulige fordeling af børn på holdet. Dit barns plads på ventelisten bliver beregnet ud fra, hvornår barnet er oprettet i systemet. Det er altså barnets anciennitet, der bestemmer pladsen på ventelisten til samtlige afdelinger, barnet er skrevet op på. Jo højere anciennitet, des bedre plads på ventelisten(erne). Bemærk at jeres position på ventelisten afgøres ud fra barnets oprettelses tidspunkt i systemet - ikke tidspunktet I melder jeg på ventelisterne nedenfor! I kan derfor frit tilmelde og framelde jer ventelisterne uden at miste en position på listen.

@mhewel mhewel changed the title Initial - tilføjet venteplads nummer Feature/794: Opskrivninger til afdeling, angiv også # på venteliste Jan 2, 2023
Copy link
Contributor

@rasmusselsmark rasmusselsmark left a comment

Choose a reason for hiding this comment

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

Det er ok med mig, vi har allerede "disclaimer" om at man ikke kan forvente plads bare fordi man står højt på venteliste

@rasmusselsmark
Copy link
Contributor

Overlapper med #807, så vi venter med denne

@rasmusselsmark rasmusselsmark marked this pull request as draft February 26, 2023 14:53
@mhewel mhewel self-assigned this Jun 13, 2023
@mhewel mhewel marked this pull request as ready for review May 26, 2024 17:05
Copy link

what-the-diff bot commented Jun 17, 2024

PR Summary

  • Enhancement in member sign-up process
    In the members/templates/members/department_signup.html file, this PR introduces an advancement where we go through all waiting lists of each child. This allows us to see which child is waiting for which department.

  • Improved waiting list matching
    It's now easier to associate children with their corresponding departments. A condition has been added to ensure that the department that a child is on the waiting list for matches the current department.

  • User-friendly display
    A neat display of a child's first name and their position on the waiting list has been added. This only happens if they're subscribed, aiding in better visualization of child and waitlist data.

  • Code Formatting and Indentation
    Non-functional changes were made as well, primarily on the code's look and readability. These include adjusting the indentation and formatting of the code to adhere to best practices.

  • Link Update
    Lastly, the 'Opskriv' button has been updated. The link now includes the IDs of both the child and department, providing a more personalized experience.

@lakridserne lakridserne merged commit de17b85 into master Jun 17, 2024
5 checks passed
@lakridserne lakridserne deleted the feature/794-Nummer-på-venteliste-under-Opskriv-til-afdelinger branch June 17, 2024 19:11
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.

[Feature]Opskrivninger til afdeling, angiv også # på venteliste (efter "X er opskrevet")
4 participants