Hi Justin Thanks for the feedback. I'll have another look at this and re-submit.
CC On Thu, Apr 8, 2010 at 1:21 AM, Justin Sherrill <[email protected]> wrote: > On 4/6/10 9:55 AM, Colin Coe wrote: >> Hi all >> >> As per the API Addition page, I've added 'arch' to the >> channel.list*Channels API calls. >> >> Comments/criticisms welcome >> >> CC >> >> >> >> >> _______________________________________________ >> Spacewalk-devel mailing list >> [email protected] >> https://www.redhat.com/mailman/listinfo/spacewalk-devel > Hey Colin, > > Having it do the lookups of package Arch in the Serializer really isn't > the right way to do it as Serializers are just meant to translate the > existing data. > > I would either add joins to rhnChannelArch to get the label in each of > the queries or write a elaborator that just gets the channel arch and > make each of those queries use the elaborator. > > Thanks, > > -Justin > > > -- > Justin Sherrill, RHCA 1801 Varsity Drive. > Software Engineer Raleigh, NC 27603 > Red Hat, Inc. > > _______________________________________________ > Spacewalk-devel mailing list > [email protected] > https://www.redhat.com/mailman/listinfo/spacewalk-devel > _______________________________________________ Spacewalk-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/spacewalk-devel
