search

phildini / logtacts

Better contact management.
https://www.contactotter.com
MIT License
60 stars 19 forks source link

Use .first() to find BookOwner in managers #85

Closed adamalton closed 8 years ago

adamalton commented 8 years ago

This avoids fetching all of the BookOwners for the given user, as only the first one is used.

codecov-io commented 8 years ago

Current coverage is 77.17% (diff: 100%)

Merging #85 into master will not change coverage

@@             master        #85   diff @@
==========================================
  Files            73         73          
  Lines          2572       2572          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1985       1985          
  Misses          587        587          
  Partials          0          0

Powered by Codecov. Last update 0ed0e55...28371eb

phildini commented 8 years ago

Thanks @adamalton! TIL about .first()