search

langrenn-sprint / event-service

Backend service to adminster events, will generate racelists etc
Apache License 2.0
0 stars 0 forks source link

Fjerne validering på ageclass #139 #141

Closed heming-langrenn closed 10 months ago

heming-langrenn commented 11 months ago

Fjernet validering - det blir feil å validere på en parameter (klasse) som er mastret av et eksternt system der vi ikke kontrollerer hvilke verdier som kan komme.

stigbd commented 11 months ago

Grunnen til denne valideringa er fordi vi seinare i logikken skal lage raceclass_name på bakgrunn av verdien i klasse. Event-service tillater kun verdiar i klasse som den er i stand til å konvertere til raceclass_name.

Sjekk https://github.com/langrenn-sprint/event-service/blob/58bf4ddda184afa028f8014601d9ab4e7525594e/event_service/commands/events_commands.py#L97-L111

Forøvrig er det ein del meir som skal til for å støtte iSonen-formatet. Eg har ein branch på gang som har fiksa det meste.

heming-langrenn commented 11 months ago

Grunnen til denne valideringa er fordi vi seinare i logikken skal lage raceclass_name på bakgrunn av verdien i klasse. Event-service tillater kun verdiar i klasse som den er i stand til å konvertere til raceclass_name.

Sjekk

https://github.com/langrenn-sprint/event-service/blob/58bf4ddda184afa028f8014601d9ab4e7525594e/event_service/commands/events_commands.py#L97-L111

Forøvrig er det ein del meir som skal til for å støtte iSonen-formatet. Eg har ein branch på gang som har fiksa det meste.

OK, you fix then :-).

stigbd commented 10 months ago

Lukker denne, sidan vi no støtter iSonen-formatet.