Department members

[JondareHM]

Fixes #391

[codecov]

Codecov Report

Merging #471 (f2a59df) into master (4b99280) will increase coverage by 0.68%.
The diff coverage is 91.01%.

@@            Coverage Diff             @@
##           master     #471      +/-   ##
==========================================
+ Coverage   69.26%   69.95%   +0.68%     
==========================================
  Files         111      119       +8     
  Lines        2834     2922      +88     
==========================================
+ Hits         1963     2044      +81     
- Misses        871      878       +7     

Impacted Files	Coverage Δ
members/admin/union_admin.py	58.82% <ø> (ø)
...management/commands/create_membership_activitys.py	0.00% <ø> (ø)
members/management/commands/get_live_data.py	0.00% <0.00%> (ø)
members/urls.py	100.00% <ø> (ø)
members/models/union.py	94.36% <89.18%> (-5.64%)	⬇️
members/migrations/0020_union_meeting_notes.py	100.00% <100.00%> (ø)
members/migrations/0021_auto_20200421_1313.py	100.00% <100.00%> (ø)
members/migrations/0022_merge_20200526_1131.py	100.00% <100.00%> (ø)
members/migrations/0023_merge_20200615_1116.py	100.00% <100.00%> (ø)
members/migrations/0024_auto_20200615_1244.py	100.00% <100.00%> (ø)
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4b99280...f2a59df. Read the comment docs.

[JondareHM]

Mangler @lakridserne's implementation af union.members, og @Rotendahl du skulle hjælpe med at begrænse medlemslisten til når man er ordentligt logget ind, men ellers er det færdigt.

Skal opdateres en smule når #455 bliver deployet, men det er ikke meget.

[Rotendahl]

Sejt at din første gang med hele MVC flow er kommet nu.
Der er lige nogle forbedringsforslag men ellers solidt arbejde!

[JondareHM]

Ved ikke hvorfor den bliver ved med at sige der er fejl, ser ud til at testsne passer helt fint.

[JakobLibak]

Det er en rigtig god start. Jeg vil gerne have en ekstra kolonne med [Tilmeldingsdato] og også gerne email på forældre om muligt. Vi skelner mellem Type Barn og Forælder. Vil Frivillig ikke være mere retvisende? @JondareHM @Rotendahl

Jeg har kun testet /union_overview/. Skriv hvis jeg skal teste andet.

[JondareHM]

Det er en rigtig god start. Jeg vil gerne have en ekstra kolonne med [Tilmeldingsdato] og også gerne email på forældre om muligt.

Med tilmeldingsdato, mener du da dato de tilmeldte sig foreningen (altså det even som gav dem medlemskab) eller dato de joinede medlemssystemet (altså den dato der bliver brugt til ventelister osv)?

Og forældre email er faktisk mere besværligt end som så, da jeg lige nu kun får Person elementer ud fra union.members() funktionen, så det ville være en hel masse flere kald til databasen for at kunne gå ind og finde de personers familie, og derefter deres forældre.

Vi skelner mellem Type Barn og Forælder. Vil Frivillig ikke være mere retvisende? @JondareHM @Rotendahl

Vi bruger 'Forældre' alle andre steder, og der er jo også situationer hvor forældre har meldt sig ind for at kunne stemme, uden at være frivillige.

Men det gør ikke den store forskel for mig, kan sagtens ændre det hvis det er.

Jeg har kun testet /union_overview/. Skriv hvis jeg skal teste andet.

Nope, det er hvad der er af nyt, sådan på frontenden.

@JakobLibak

[JakobLibak]

Hurtig kommentar fra frokostkø:

Tilmeldingsdato til aktiviteten, der gav medlemsskab.

Drop forældremail for nu.

Behold resten som det er.
@JondareHM

[JondareHM]

Det er desværre samme historie: Enten skal jeg ændre hvad der sendes ud fra members(), hvilket ville gøre den funktion mindre generelt brugbar, eller også skal der laves en hel masse flere database kald.

Jeg kan tilføje alt hvad der findes i selve Person modellen, men alt udover det ryger desværre ind i de her problemer.

(Og det er ikke at de ikke er mulige, det er bare et spørgsmål om hvorvidt vi vil acceptere de tradeoffs som de indebærer)

[rasmusselsmark]

Vi har talt om denne i dag i udviklingsgruppen, og har besluttet at lukke dette pull request, da @lakridserne er ved at forberede ændringer til medlemstabel, som vil gøre denne feature enklere at implementere.

Beklager, og tak for indsatsen