Code review comment for lp://staging/~ptressel/sahana-eden/devel

Revision history for this message
nursix (nursix.org) wrote :

If necessary, re-implement this in a more recent version of CRUD.

Basically that would mean to just extend S3CRUD.get_list_fields, whereas the extra field pre-loading is already implemented. The S3VirtualField class doesn't seem to be necessary anymore.

review: Disapprove

« Back to merge proposal