While playing around with existing Achievement routes I stumbled upon this user filter that didn't quite work. The filter seems to expect a numeric ID and corresponding user id field, but looking at current user model/migrations there is only a uuid field present. Updated it to expect UUID values and use uuid field for lookup.
Example of previously broken lookup: /api/achievements/achievements/?user=c3961109-045c-43bd-a4d6-329209accddf (and providing "legacy" numeric id fails because of missing DB column)
Literally (as far as I can remember) my first PR to a Django project ever, so just trying to get my toes wet 😄
While playing around with existing Achievement routes I stumbled upon this user filter that didn't quite work. The filter seems to expect a numeric ID and corresponding user
id
field, but looking at current user model/migrations there is only auuid
field present. Updated it to expect UUID values and useuuid
field for lookup.Example of previously broken lookup:
/api/achievements/achievements/?user=c3961109-045c-43bd-a4d6-329209accddf
(and providing "legacy" numeric id fails because of missing DB column)Literally (as far as I can remember) my first PR to a Django project ever, so just trying to get my toes wet 😄