concrexit issueshttps://gitlab.science.ru.nl/thalia/concrexit/-/issues2017-11-22T15:51:50+01:00https://gitlab.science.ru.nl/thalia/concrexit/-/issues/452Valideer dat mensen met automatische incasso een IBAN hebben2017-11-22T15:51:50+01:00Joost Rijneveldjoost@joostrijneveld.nlValideer dat mensen met automatische incasso een IBAN hebben### One-sentence description
A member can now be set to be paying by direct debit, but not have a valid IBAN.
### Current behaviour
A member can now be set to be paying by direct debit, but not have a valid IBAN. This creates empty re...### One-sentence description
A member can now be set to be paying by direct debit, but not have a valid IBAN.
### Current behaviour
A member can now be set to be paying by direct debit, but not have a valid IBAN. This creates empty records in !503.
### Expected behaviour
Members should only be allowed to pay by direct debit if they have a valid IBAN.
### Steps to reproduce
1. Set a member's IBAN to the empty string
2. Edit a member to say that they pay by direct debit
3. Observe that this is possiblehttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/448Off-by-one in `_send_queue_mail(event)`2017-12-10T21:29:26+01:00Sébastiaan VersteegOff-by-one in `_send_queue_mail(event)`### One-sentence description
Off-by-one in `_send_queue_mail(event)`
### Current behaviour
We sturen nu een mailtje naar de persoon die na het cancellen als eerste in de wachtrij staat, maar we moeten een mailtje sturen naar de persoo...### One-sentence description
Off-by-one in `_send_queue_mail(event)`
### Current behaviour
We sturen nu een mailtje naar de persoon die na het cancellen als eerste in de wachtrij staat, maar we moeten een mailtje sturen naar de persoon die voor het cancellen als eerste in de wachtrij staat.
### Expected behaviour
We moeten een mailtje sturen naar de persoon die voor het cancellen als eerste in de wachtrij staat.
### Steps to reproduce
1. Create event, add registrations
2. Cancel registration in frontend
### How to fix
In `_registration_cancel` moeten we eerst `_send_queue_mail(event)` doen, en dan pas de `reg.date_cancelled` op `timezone.now()` zetten.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/447Send queue mail wordt verzonden als iemand op de wachtrij zich afmeldt2017-07-05T19:24:09+02:00Sietse MoorenSend queue mail wordt verzonden als iemand op de wachtrij zich afmeldt### One-sentence description
Send queue mail wordt verzonden als iemand op de wachtrij zich afmeldt
### Current behaviour
Als iemand in de wachtrij zichzelf er vanaf haalt wordt `_send_queue_mail` nog steeds uitgevoerd
### Expe...### One-sentence description
Send queue mail wordt verzonden als iemand op de wachtrij zich afmeldt
### Current behaviour
Als iemand in de wachtrij zichzelf er vanaf haalt wordt `_send_queue_mail` nog steeds uitgevoerd
### Expected behaviour
Als iemand in de wachtrij zichzelf er vanaf haalt wordt er geen mail verstuurd
### Steps to reproduce
1. Maak een event met max participants
2. Voeg aantal registrations toe
3. Cancel registration in de frontend van een van de gebruikers
4. Zie verzonden mail
### Original text
Je krijgt een mail dat je in de wachtlijst stond maar nu mag gaan. Als je vervolgens naar de link gaat die in de mail staat sta je nog steeds in de wachtrij. Wat moet ik nu geloven.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/446&nbsp ipv een spatie bij aankomende evenementen2017-07-05T19:24:09+02:00Ghost User  ipv een spatie bij aankomende evenementen### One-sentence description
Er staat ` ` in plaats van een spatie in het tekstblokje wat de tekst van het evenement laat zien. Op de pagina van het evenement zelf is het echter wel een spatie zoals hieronder te zien.
![Screensh...### One-sentence description
Er staat ` ` in plaats van een spatie in het tekstblokje wat de tekst van het evenement laat zien. Op de pagina van het evenement zelf is het echter wel een spatie zoals hieronder te zien.
![Screenshot_4](/uploads/5cd7d4fa54e046c095e260252091058f/Screenshot_4.png)
### Current behaviour
Er staat ` ` bij ALV;
![Screenshot_2](/uploads/1b971d25e160d963f1dfa38910851057/Screenshot_2.png)
Ook op de Engelse pagina bij end of semester party:
![Screenshot_3](/uploads/f3495e88985d598f09b08c5728b7782e/Screenshot_3.png)
### Expected behaviour
Ik zou er graag een spatie zien staan
### Steps to reproduce
1. Open thalia.nu
2. Scroll naar beneden naar "Aankomende evenementen"
https://gitlab.science.ru.nl/thalia/concrexit/-/issues/443Events api for calendarjs no longer accisible for unauthorized users2017-12-10T21:29:25+01:00Luuk ScholtenEvents api for calendarjs no longer accisible for unauthorized users### One-sentence description
Unauthorized users can no longer execute api call to calendarjs api.
### Current behaviour
There are no more events on the calendar if you are not authenticated. This is because the api results in a 403.
##...### One-sentence description
Unauthorized users can no longer execute api call to calendarjs api.
### Current behaviour
There are no more events on the calendar if you are not authenticated. This is because the api results in a 403.
### Expected behaviour
There should be events on the calendar if you are not authenticated
### Steps to reproduce
1. Go to `beta.thalia.nu`
2. Go to calender
3. Open the developer console
4. See that the api results in a 403https://gitlab.science.ru.nl/thalia/concrexit/-/issues/439Former members are listed confusingly in a committee's admin2018-01-17T20:01:39+01:00Thom WiggersFormer members are listed confusingly in a committee's admin### One-sentence description
The committee pages should only list current members
### Current behaviour
Current and former members are listed, which may be confusing.
### Expected behaviour
Only list current members.
##...### One-sentence description
The committee pages should only list current members
### Current behaviour
Current and former members are listed, which may be confusing.
### Expected behaviour
Only list current members.
### Steps to reproduce
1. View a committee with former and current members in the admin
2. See all of themSébastiaan VersteegSébastiaan Versteeghttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/435Event dates are missing from plaintext newsletter2017-06-01T12:34:11+02:00Sébastiaan VersteegEvent dates are missing from plaintext newsletter### One-sentence description
Event dates are missing from plaintext newsletter
### Current behaviour
All places where a date should be in the text there is nothing.
### Expected behaviour
There should be dates
### Steps to reproduc...### One-sentence description
Event dates are missing from plaintext newsletter
### Current behaviour
All places where a date should be in the text there is nothing.
### Expected behaviour
There should be dates
### Steps to reproduce
1. Send and receive a newsletter
2. Look at the plaintext versionhttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/428Photo gallery previews are not resized correctly2017-12-10T21:11:17+01:00Sébastiaan VersteegPhoto gallery previews are not resized correctly### One-sentence description
Photo gallery previews are not resized correctly
### Current behaviour
![Screen_Shot_2017-05-04_at_11.27.12](/uploads/bc60426ffe2534499a5cf5b6ade0de7b/Screen_Shot_2017-05-04_at_11.27.12.png)
### Expected ...### One-sentence description
Photo gallery previews are not resized correctly
### Current behaviour
![Screen_Shot_2017-05-04_at_11.27.12](/uploads/bc60426ffe2534499a5cf5b6ade0de7b/Screen_Shot_2017-05-04_at_11.27.12.png)
### Expected behaviour
![Screen_Shot_2017-05-04_at_11.55.08](/uploads/fa5b7567885f164a3cc92a9eda8be07f/Screen_Shot_2017-05-04_at_11.55.08.png)
### Steps to reproduce
1. Upload 1024h768w photo
2. Look up 1024w768h thumbnail.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/425Meerdere albums kunnen dezelfde slug hebben2017-12-10T21:11:17+01:00Tom van BusselMeerdere albums kunnen dezelfde slug hebben### One-sentence description
Meerdere albums met dezelfde slug kunnen aangemaakt worden.
### Current behaviour
Als je een album aanmaakt met een slug die al door een ander album gebruikt wordt, zijn er twee albums met dezelfde slug. A...### One-sentence description
Meerdere albums met dezelfde slug kunnen aangemaakt worden.
### Current behaviour
Als je een album aanmaakt met een slug die al door een ander album gebruikt wordt, zijn er twee albums met dezelfde slug. Als je vervolgens naar de pagina van het album gaat (op de website, niet in de admin-panel) krijg je een ``MultipleObjectsReturned`` error.
### Expected behaviour
De admin panel zou een error moeten geven als een album met een al gebruikte slug wordt geüpload.
### Steps to reproduce
1. Upload een album met de slug "henk".
2. Upload nog een album met de slug "henk".
3. Ga naar foto's pagina en selecteer een van de twee albums met de slug "henk".
???
4. ErrorTom van BusselTom van Busselhttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/423HTML characters not escaped properly in vacancies2017-12-10T21:29:49+01:00Milan van StiphoutHTML characters not escaped properly in vacancies### One-sentence description
&nbsp's are visible in vacancies.
### Current behaviour
See above.
### Expected behaviour
Not showing &nbsp's.
### Steps to reproduce
1. visit https://thalia.nu/career/vacancies
2. Ctrl+F "nbsp"### One-sentence description
 's are visible in vacancies.
### Current behaviour
See above.
### Expected behaviour
Not showing  's.
### Steps to reproduce
1. visit https://thalia.nu/career/vacancies
2. Ctrl+F "nbsp"https://gitlab.science.ru.nl/thalia/concrexit/-/issues/421Foto's vergroot weergeven gedeeld album2017-05-10T20:51:40+02:00Frank GerlingsFoto's vergroot weergeven gedeeld album### One-sentence description
Foto's groot weergeven bij gedeeld album gaat niet.
### Current behaviour
Bij zowel de "open" als de Thalia-eigen versie van het ouderdagfotoalbum geeft het klikken op een foto de volgende foutmelding:
![...### One-sentence description
Foto's groot weergeven bij gedeeld album gaat niet.
### Current behaviour
Bij zowel de "open" als de Thalia-eigen versie van het ouderdagfotoalbum geeft het klikken op een foto de volgende foutmelding:
![2017-04-29-154813_822x389_scrot](/uploads/319664c7a84f1bd9b6e3786f71d3c546/2017-04-29-154813_822x389_scrot.png)
![2017-04-29-154759_850x390_scrot](/uploads/557b7bd01a4a31bf2cfc2d4bc7c9af71/2017-04-29-154759_850x390_scrot.png)
### Expected behaviour
Bij het klikken op een foto een vergrootte weergave als pop-up tonen.
### Steps to reproduce
1. Het fotoalbum van de ouderdag
2. Klik op een foto
3. Krijg foutmeldinghttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/419Tekst bij event als je op de wachtlijst staat is verwarrend2017-05-03T20:06:33+02:00Sébastiaan VersteegTekst bij event als je op de wachtlijst staat is verwarrend### One-sentence description
Verwarrende tekst als je op de wachtlijst staat.
### Current behaviour
De tekst is 'Afmelden is niet meer mogelijk zonder de volledige kosten van €5,00 te betalen. Let op: je kunt je hierna niet meer aanme...### One-sentence description
Verwarrende tekst als je op de wachtlijst staat.
### Current behaviour
De tekst is 'Afmelden is niet meer mogelijk zonder de volledige kosten van €5,00 te betalen. Let op: je kunt je hierna niet meer aanmelden.' en dat is verwarrend.
### Expected behaviour
Deze tekst moet niet getoond worden want hij is niet van toepassing.
### Steps to reproduce
1. Zet jezelf op de wachtlijst voor een evenement
2. Zorg dat de cancel deadline is verlopen
3. Zie de teksthttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/418Profiel pagina gebruikt user id als member id2017-12-10T21:29:48+01:00Thom WiggersProfiel pagina gebruikt user id als member id**[bug]**
### One-sentence description
Als de user id en de member-id niet overeen komen wordt de verkeerde gebruiker weergegeven
### Current behaviour
De userid (zoals in de link van de 'view profile' link onder `/account/`) wordt geb...**[bug]**
### One-sentence description
Als de user id en de member-id niet overeen komen wordt de verkeerde gebruiker weergegeven
### Current behaviour
De userid (zoals in de link van de 'view profile' link onder `/account/`) wordt gebruikt voor links, maar vervolgens ga je naar `/members/profile/1/` en die levert de **Member** met `pk=1` op.
### Expected behaviour
Be consistent, use the user id.
### Steps to reproduce
0. Create some users with member-information, make sure their IDs **do not** match
1. Go to `/account/`
2. Click "View public profile"
3. See a *different* profile.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/417Lidmaatschapsdata van een commissie kunnen buiten het bestaan van een commiss...2017-12-10T21:12:51+01:00Thom WiggersLidmaatschapsdata van een commissie kunnen buiten het bestaan van een commissie vallen**[bug]**
### One-sentence description
Lidmaatschapsdata van een commissie kunnen buiten het bestaan van een commissie vallen
### Current behaviour
Als je een commissielid bewerkt, kan die lid zijn van een commissie die volgens de dat...**[bug]**
### One-sentence description
Lidmaatschapsdata van een commissie kunnen buiten het bestaan van een commissie vallen
### Current behaviour
Als je een commissielid bewerkt, kan die lid zijn van een commissie die volgens de data op de website toen niet bestond.
### Expected behaviour
Dat zou niet mogen.
### Steps to reproduce
1. Maak een commissie die bestaat tot gisteren
2. Maak iemand vanaf vandaag lid van de commissie
3. dat werkt.Bram in 't ZandtBram in 't Zandthttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/416Board membership cannot have end dates2019-04-12T23:42:09+02:00Thom WiggersBoard membership cannot have end dates### One-sentence description
Board members are currently not allowed to have an end date for their group membership even though we might want them to have one. Examples are the boards of 2012-2013 and 2017-2018.
### Current behaviour
1...### One-sentence description
Board members are currently not allowed to have an end date for their group membership even though we might want them to have one. Examples are the boards of 2012-2013 and 2017-2018.
### Current behaviour
1. Try to enter an end date for a board membership.
2. Does not work
### Expected behaviour
Should workTobias van der WerffTobias van der Werffhttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/415Besturen maken en bewerken is defect als er geen einddatum is ingevuld2017-12-10T21:29:47+01:00Thom WiggersBesturen maken en bewerken is defect als er geen einddatum is ingevuld**[bug]**
### One-sentence description
Het maken van besturen is stuk als je geen einddatum invult, `validate_unique` kan niet handelen dat `until` `None` is.
### Current behaviour
Crash
### Expected behaviour
Gemaakt bestuu...**[bug]**
### One-sentence description
Het maken van besturen is stuk als je geen einddatum invult, `validate_unique` kan niet handelen dat `until` `None` is.
### Current behaviour
Crash
### Expected behaviour
Gemaakt bestuur
### Steps to reproduce
1. Probeer een bestuur te maken met alleen een begindatum of geen van beide
2. Huil.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/405Commissielidmaatschap zonder `member`?2017-12-10T21:11:16+01:00Joost Rijneveldjoost@joostrijneveld.nlCommissielidmaatschap zonder `member`?We ontvingen een mailtje van @yaarts dat 'ie probeerde iemand voorzitter te maken van een commissie precies op 't moment dat we de onderstaande error logde;
```
ERROR (EXTERNAL IP): Internal Server Error: /admin/activemembers/committ...We ontvingen een mailtje van @yaarts dat 'ie probeerde iemand voorzitter te maken van een commissie precies op 't moment dat we de onderstaande error logde;
```
ERROR (EXTERNAL IP): Internal Server Error: /admin/activemembers/committee/31/change/
Internal Server Error: /admin/activemembers/committee/31/change/
RelatedObjectDoesNotExist at /admin/activemembers/committee/31/change/
CommitteeMembership has no member.
[knip (zie #django-errors voor volledige traceback)]
File "./activemembers/models.py" in validate_unique
266. member=self.member))
```
Vreemd genoeg slaagde een tweede poging wel.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/394Shared thumbnails are never created2018-10-14T20:22:23+02:00Thom WiggersShared thumbnails are never createdAls je een album shareable maakt, werkt thumbnail-generatie niet meer. Dat levert op dat je dus gebroken thumbnails krijgt voor thumbnails _die nog niet gemaakt zijn_.Als je een album shareable maakt, werkt thumbnail-generatie niet meer. Dat levert op dat je dus gebroken thumbnails krijgt voor thumbnails _die nog niet gemaakt zijn_.https://gitlab.science.ru.nl/thalia/concrexit/-/issues/373Events without organisers break2018-11-25T01:45:55+01:00Joost Rijneveldjoost@joostrijneveld.nlEvents without organisers breakGerelateerd aan #357 en !418. Als je event geen organiser heeft (wat waarschijnlijk alleen voor komt bij gemigreerde events uit concrete5) wordt er een `.pk` van een `None` bekeken, en dat gaat stuk. Twee opties: ofwel even met de hand e...Gerelateerd aan #357 en !418. Als je event geen organiser heeft (wat waarschijnlijk alleen voor komt bij gemigreerde events uit concrete5) wordt er een `.pk` van een `None` bekeken, en dat gaat stuk. Twee opties: ofwel even met de hand een organiser aanklikken bij alle oude events (dit heeft denk ik mijn voorkeur omdat het het verplicht zijn van een organiser handhaaft, maar is wat moeite), ofwel geen permissions checken als er geen organiser is. In dat laatste geval is dit een hotfix waard.Thom WiggersThom Wiggershttps://gitlab.science.ru.nl/thalia/concrexit/-/issues/371Newsletter ordering only possible for info or events seperately2019-05-15T19:55:26+02:00Luuk ScholtenNewsletter ordering only possible for info or events seperately### One-sentence description
Newsletter ordering only possible for info or events seperately
### Current behaviour / Reproducing the bug
1. Add a newsletter
2. Add content and events
3. Try to mix them in the order
4. No mixing in the...### One-sentence description
Newsletter ordering only possible for info or events seperately
### Current behaviour / Reproducing the bug
1. Add a newsletter
2. Add content and events
3. Try to mix them in the order
4. No mixing in the result
### Expected behaviour
1. Add a newsletter
2. Add content and events
3. Try to mix them in the order
4. Result is correct