Optimise the trello:load_people task

Only update the DB if the card has changed

Author: lylo

README.md

@@ -45,6 +45,8 @@ trello:
   token: [YOUR API TOKEN]
 ```
 
+You also need to set the environment variable `TRELLO_BOARD_ID` (e.g. `Yae2dtTq`).
+
 ### Configure Google Sign In
 
 You need to configure the app with Google creds.

lib/tasks/trello.rake

@@ -1,6 +1,4 @@
 namespace :trello do
-  WHOS_WHO = "Xau2Ttaq"
-
   desc "Looks for Trello cards that have been archived (closed) and discards the person in the database"
   task clean: :environment do
     Rails.logger = Logger.new(STDOUT)
@@ -22,7 +20,10 @@ namespace :trello do
     Rails.logger = Logger.new(STDOUT)
     Rails.logger.info "Loading cards from Trello"
 
-    lists = Trello::Board.find(WHOS_WHO).lists
+    board = ENV['TRELLO_BOARD_ID']
+    raise "Missing environment variable TRELLO_BOARD_ID" if board.nil?
+
+    lists = Trello::Board.find(board).lists
     lists.each do |list|
       Rails.logger.info "Checking #{list.name}"
 
@@ -33,6 +34,11 @@ namespace :trello do
 
         Rails.logger.info "checking #{card.name}"
 
+        if person.updated_at > card.last_activity_date
+          Rails.logger.info "No activity. Skipping"
+          next
+        end
+
         name, title = card.name.split /[\[(]/
         person.name = name.strip if name
         person.title = title.gsub(/[\)\]]/, "").strip if title