Performance improvement for /v2/badges

mozilla / openbadger

badger badger badger badger

Mozilla Public License 2.0

34 stars 23 forks source link

Performance improvement for /v2/badges #290

Closed ghost closed 11 years ago

ghost commented 11 years ago

This just removes the claimCodes field from the api.badges function's query. The data was completely unused, and stripped out moments later in the normalizeBadge() function. Excluding it from the query is a massive performance increase when badges contain lots of claimCodes.

cmcavoy commented 11 years ago

looks good to me. can you merge and manage the release?

ghost commented 11 years ago

I can indeed.