Merge pull request #134 from dalthon/include-associations-on-api

Uses "includes" to avoid doing too much queries to database

Author: mlitwiniuk

.travis.yml

@@ -9,7 +9,8 @@ gemfile:
   - gemfiles/rails_5.1.gemfile
   - gemfiles/rails_5.2.gemfile
 services:
- - redis-server
+  - postgresql
+  - redis-server
 env:
   - LIT_STORAGE=hash
   - LIT_STORAGE=redis
@@ -18,4 +19,3 @@ before_script:
   - psql -c 'create database lit_test;' -U postgres
   - RAILS_ENV=test bundle exec rake db:migrate
 script: "bundle exec rake test"
-

app/controllers/lit/api/v1/localizations_controller.rb

@@ -22,12 +22,17 @@ def last_change
     private
 
     def fetch_localizations
+      scope = Localization.includes(
+        :locale,
+        :localization_key
+      )
+
       if params[:after].present?
         after_date = Time.parse("#{params[:after]} #{Time.zone.name}")
                          .in_time_zone
-        Localization.after(after_date).to_a
+        scope.after(after_date).to_a
       else
-        Localization.all
+        scope.all
       end
     end
   end