UiL-OTS-labs / ppn-backend

Backend application for the adult participant administration system
https://experimenten-admin.lab.hum.uu.nl
Other
0 stars 0 forks source link

pp kan zich twee keer opgeven voor mailinglist #114

Open irisopgithub opened 2 years ago

irisopgithub commented 2 years ago

Dit is een beetje een raadsel voor mij, want volgens mij heb ik dit getest en kreeg ik toen de tweede keer een melding dat ik al op de lijst stond, en geen tweede entry in de database.

Iig, er is nu een pp in de database die zich twee keer heeft opgegeven, ik zie geen verschil tussen de aanmeldingen. pp nummer 3004 en 3005. Zal rond 21.45h gebeurd zijn.

Als ik op de acceptatieserver me twee keer met dezelfde mail aanmeld nu, krijg ik geen melding dat ik al op de lijst sta, maar ook geen tweede entry in de database.

tymees commented 2 years ago

Dit is wel heel erg raar ja...

Ik dacht dat het misschien een probleem was dat het verzoek 2x te snel achter elkaar ingediend werd, en dat poging 1 nog niet in de DB stond toen pogin 2 ging kijken. Echter, in de logs zie ik genoeg tijd zitten tussen de twee verzoeken dat dit niet kan.

En als ik nu de app vraag of de emails hetzelfde zijn zegt ie 'ja', dus ik kan me echt niet bedenken hoe deze erdoorheen geslopen is.

Als ik op de acceptatieserver me twee keer met dezelfde mail aanmeld nu, krijg ik geen melding dat ik al op de lijst sta, maar ook geen tweede entry in de database.

Klopt, de code vind het prima als je je nogmaals op de mailinglist wil zetten. Er gebeurd dan eigenlijk niks.

Als je toch zo'n melding wil hebben kan dat, het is maar 2 regels code meer.

irisopgithub commented 2 years ago

gek toch? zou het kunnen dat hij 1 van de keren er een spatie achter had getiept ofzo, die er later afgestript is? (dat op het moment van de check ze niet hetzelfde waren, maar hoe ze uiteindelijk in de database terecht kwamen wel)

ik wil wel zo'n melding hebben, dat staat wel professioneel vind ik. dus als het weinig werk is, heel graag!

tymees commented 2 years ago

gek toch? zou het kunnen dat hij 1 van de keren er een spatie achter had getiept ofzo, die er later afgestript is? (dat op het moment van de check ze niet hetzelfde waren, maar hoe ze uiteindelijk in de database terecht kwamen wel)

Dat heb ik ook gecheckt, maar tijdens de check houd hier rekening mee. Beide kanten van de vergelijking worden omgezet naar lowercase en worden de spaties van weggestript.

ik wil wel zo'n melding hebben, dat staat wel professioneel vind ik. dus als het weinig werk is, heel graag!

Done!

irisopgithub commented 2 years ago

weer een pp die twee keer op de maillijst staat: 3071 en 3072. identiek, voor zover ik kan zien.

irisopgithub commented 2 years ago

wil je nog iets bekijken aan die dubbele ppn (3071/3072 en 3004/3005)? anders haal ik er bij elk eentje weg namelijk. tnx!

tymees commented 2 years ago

3004/3005 kan er eentje weg, de andere moet even wachten tot donderdag. Dan heb ik de tijd om naar die te kijken, vandaag alleen vergaderingen....

irisopgithub commented 2 years ago

ok, ik wacht wel ff met allebei.

Misschien gerelateerd: er is een comment in het systeem (comment 1182) dat voor mij suggereert dat ppn vrij makkelijk snel twee keer achter elkaar op 'inschrijven' kunnen klikken en daardoor problemen krijgen - deze pp had de melding gekregen dat ze niet mee kon doen omdat ze al mee had gedaan (wat niet zo was), en was daarom niet komen opdagen. Ze stond wel gewoon ingeschreven, had blijkbaar de bevestigingsmail niet gezien, en had geen reminder-mail gehad omdat ze zich de avond of ochtend voor de afspraak pas had ingeschreven.

Ik heb ook een paar mailtjes gehad de afgelopen weken van ppn met meldingen waardoor ik dacht dat een pp twee keer snel op inschrijven had kunnen drukken. (of misschien is het niet twee keer klikken van de pp maar dat het systeem 'hikt'.)

Ik weet niet of dit op te lossen is - in het oude systeem kwam het iig niet voor.

irisopgithub commented 2 years ago

Hier is nog zo'n melding, uit een mailtje dat ik kreeg van een (andere) pp:

"Even checken: heb ik mij opgegeven voor het experiment Verhalen en Emotie 4? Ik kreeg een melding dat het tijdstip al bezet was maar kon me daarna niet voor een ander tijdstip opgeven." d.d. van de mail is 27 juni, 20.35h. (terzijde, maar deze pp had de bevestigingsmail niet gezien omdat ze per ongeluk een ander mailadres had gebruikt dan ze had willen gebruiken, ik heb die daarna ook nog gemerged)

tymees commented 2 years ago

dat voor mij suggereert dat ppn vrij makkelijk snel twee keer achter elkaar op 'inschrijven' kunnen klikken en daardoor problemen krijgen

Ja ik zie vergelijkbare signalen in de logs... Het inschrijven is wat traag wel (een paar sec), dus ik vermoed dat ze ongeduldig zijn.

Ik ga kijken of 1) ik de knop helemaal kan disablen na de eerste klik en 2) of ik visueel wat duidelijker kan maken dat er wel wat gebeurd

tymees commented 2 years ago

Ik heb nu 2 manieren dat opnieuw inschrijven kon gebeuren afgedekt, bij zowel inschrijven voor een experiment als voor het maken van een account/aanmelden voor de mailinglist.

Hopelijk helpt dit wat, maar aangezien ik niet snap hoe iemand dubbel op de mailinglist kan komen weet ik niet zeker of dit alles oplost.

Komt later vandaag iig online

irisopgithub commented 2 years ago

ik ben bang dat dit niet goed heeft uitgepakt, krijg van tijn de volgende melding:

"Ik heb de laatste dagen van meerdere mensen gehoord dat het niet lukt om het inschrijfformulier voor het experiment te versturen. Het lijkt iets met de browser te maken te hebben, want in Firefox werkt het wel, maar in Chrome, Safari en Internet Explorer niet. Ik heb het zelf ook geprobeerd maar bij mij werkte het ook alleen in Firefox. Weet jij wat er aan de hand is?"

bij mij werkt het in chrome ook niet - dwz, er blijft eindeloos een rondgaand dingetje rondgaan in de 'schrijf je in' knop.

irisopgithub commented 2 years ago

krijg nu van li een gelijksoortige melding van een andere pp

tymees commented 2 years ago

Ik heb een hotfix online gezet die het voor alle browsers zou moeten fixen. Morgen komt 'nette' fix.

tymees commented 2 years ago

Achtergrond info: FF is de enige die event priority rules volgens spec heeft geimplementeerd blijkbaar, waardoor die netjes eerst de submit deed en daarna pas de disable van de submit knop. De rest, doet dat dus andersom....

De hotfix forceert de submit eerst, zodat ie overal goed gaat

irisopgithub commented 2 years ago

het werkt bij mij nog steeds niet in chrome en safari.

die achtergrond - mijn hemel! zo kan een mens toch niet werken!

tymees commented 2 years ago

het werkt bij mij nog steeds niet in chrome en safari.

Er zat nog een cache op de server roet in het eten te gooien. (Deze fix was erg hot, niet via de normale manier, vandaar...) Het kan zijn dat je nog een lokale cache hebt, fresh reload (cmd+shift+r volgensmij) helpt daar bij ;)

die achtergrond - mijn hemel! zo kan een mens toch niet werken!

Tis al een stuk beter nu we geen IE meer hoeven te ondersteunen nu die officieel dood is ;)

irisopgithub commented 2 years ago

ah yes, hij doet het nu. dankjewel!!! (op je vrije avond ook nog)