Use a simplier serializer and do the course counts as a single aggregate...

... query
I'd prefer clients not be aware of the serializer implementations -- since
Apros is the only client so far I think it's okay that we've done the
cursory review

On Mon, Dec 29, 2014 at 2:01 PM, chrisndodge <notifications@github.com>
wrote:

> In lms/djangoapps/organizations/views.py
> <https://github.com/edx-solutions/edx-platform/pull/363#discussion-diff-22323633>
> :
>
> >              response_data = []
> >              if users:
> >                  for user in users:
> > -                    serializer = UserSerializer(user)
> > +                    serializer = SimpleUserSerializer(user)
>
> I did a search in Apros and I did not find any other consumers of this API
> call except for the one. I could add an optional parameter to specify which
> serializer to use....
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/edx-solutions/edx-platform/pull/363/files#r22323633>.
>



-- 
Matthew Drayer
Solutions Architect
edX | MIT
mattdrayer@edx.org