search

curtisullerich / attendance

An attendance application built for the Iowa State University Cyclone Football 'Varsity' Marching Band.
MIT License
6 stars 2 forks source link

Can't delete certain absences #243

Closed danstiner closed 10 years ago

danstiner commented 10 years ago

Probably several other UI actions are also broken by this, the crux of the issue is that we are parsing datastore keys as 32bit integers. Recently Google seems to be giving out super long keys that no longer fit in 32bits, hence actions against particular keys, like an absence delete, fail.

danstiner commented 10 years ago

Looks like it was only a problem for delete absences, Integer.parseInt isn't used for keys anywhere else, we tend to use strings for keys.

curtisullerich commented 10 years ago

Awesome. Did you deploy to prod or dev?

danstiner commented 10 years ago

Prod

doitlive

curtisullerich commented 10 years ago

+1