newsdev/nyt-fec

top donor sort is slow

rshorey opened this issue · 2 comments

should we pre-compute totals and put on the donor model?

they are pre-populated but only when donor is saved. NOT when the transaction is modified. We are bypassing transaction save when we load them because of bulk_create, so I think it is safe to modify save() on ScheduleA to save the donor if it is added, but should be tested a lot for performance.

modified save in like the ugliest possible way.

whatever works bro