rails-contact 0.1.7 → 0.1.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 48540c1897658b886292f1fafe8157fa2aa7fed817d9b9ebb64213456ac958e3
-  data.tar.gz: 3329ba043c33922528e3736e8a77d739a71151c30d471bc300ef832131ab09fd
+  metadata.gz: 50cd2c0740daf7405d8085d2956f810cf0bf8883388ff12afb72ee07a6497b6c
+  data.tar.gz: 64db2836ff0701a348e215ea294dae2cef1326ef3fb633a4a69676c4b5e477ef
 SHA512:
-  metadata.gz: 34f85c62bf91b52739ad8833e884a44ac142c44b507eda5c1c1044320cda3fba787c1d20afa3d4eb196df81cf02dd806b593ef1b8d06d383790741bd80b055e1
-  data.tar.gz: 4ddb497c06c1fc0fed8ad0b5a636f5ff640afe425e1d1fe0b40c2acc9ab9500bc0b473eab23be1bde9eacfb29c972b63bf3e10095db7ed2f19cd3bdf169d7bbb
+  metadata.gz: 6b8653a3e39f49d9d6aa1bb04463df1f4b5b10ac31d759af6b5bb465b8b2b4ecfbf9a83ee8cbf60a70296d805b5df91e942031ea85f0d530075d00e18bb5a4bc
+  data.tar.gz: e42099d4322b591d5ae7646560afda7178e5ce1674512efab58ef172d34fe74b99b46583553e6cad43ff2a47c96cf95efe34bdd9d2e68958fc77a3a164585812

data/CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## 0.1.11
+
+- Default **Google sync panel** back on the engine index via `_google_sync_panel` when `google_sync_enabled` and `google_sync_ui_on_index` (default true). Host apps can override the partial or disable with `google_sync_ui_on_index = false`.
+
+## 0.1.10
+
+- Short-lived: engine index omitted the Google panel (use **0.1.11** instead).
+
+## 0.1.9
+
+- `POST /google_sync_rolling_window` → `ContactsController#google_sync_rolling_window` → `GoogleSyncJob` (re-sync rolling window: creates + updates).
+
+## 0.1.8
+
+- Contacts index: paginated search (total count, previous/next, page indicator) with Elasticsearch and database backends.
+- `POST /google_sync_unsynced` and **Sync … not yet in Google** button when sync is enabled; `GoogleSyncUnsyncedJob` runs `SyncService#sync_unsynced!` for contacts with no `google_resource_name`.
+
 ## 0.1.7
 
 - Google People API: `updateContact` sends required `updatePersonFields`; update payloads include `resourceName` and `etag`; sync preloads emails, phones, and addresses.

data/README.md

@@ -134,6 +134,22 @@ Rails::Contact.configure do |config|
 end
 ```
 
+### Google sync UI: gem default vs host override
+
+**Recommendation:** keep the **default panel in the gem** so every app gets working buttons when `google_sync_enabled` is true. You avoid copy-paste and stay aligned with new endpoints.
+
+- The index template renders `rails/contact/_google_sync_panel` when `google_sync_ui_on_index` is **true** (default).
+- **Customize without forking the engine:** add `app/views/rails/contact/_google_sync_panel.html.erb` in the host app; Rails resolves that file instead of the gem’s partial.
+- **Hide the default panel:** set `config.google_sync_ui_on_index = false` and render your own UI anywhere (same `POST` targets below).
+- **Custom index only:** override `rails/contact/index` and `<%= render "rails/contact/google_sync_panel" %>` wherever it fits (e.g. after a CSV upload section).
+
+Endpoints (used by the default partial):
+
+- `google_sync_rolling_window_contacts_path` — `GoogleSyncJob` (rolling window re-sync).
+- `google_sync_unsynced_contacts_path` — `GoogleSyncUnsyncedJob` (contacts with no `google_resource_name`).
+
+`ContactsController#index` sets `@google_contacts_pending_sync` when sync is enabled.
+
 ---
 
 ## Rake tasks

data/app/controllers/rails/contact/contacts_controller.rb

@@ -10,10 +10,41 @@ module Rails
 
       def index
         @query = params[:q].to_s.strip
-        @contacts = Search::Query.new(@query, filters: normalized_filters(filter_params)).call
+        result = Search::Query.new(
+          @query,
+          filters: normalized_filters(filter_params),
+          page: params[:page]
+        ).call
+        @contacts = result.records
+        @total_count = result.total_count
+        @page = result.page
+        @per_page = result.per_page
+        @total_pages = result.total_pages
+        @google_contacts_pending_sync = google_pending_sync_count
         render "rails/contact/index"
       end
 
+      def google_sync_unsynced
+        unless Rails::Contact.configuration.google_sync_enabled
+          redirect_to contacts_path, alert: "Google Contacts sync is disabled."
+          return
+        end
+
+        GoogleSyncUnsyncedJob.perform_later
+        redirect_to contacts_path, notice: "Google sync has been queued for contacts not yet linked."
+      end
+
+      def google_sync_rolling_window
+        unless Rails::Contact.configuration.google_sync_enabled
+          redirect_to contacts_path, alert: "Google Contacts sync is disabled."
+          return
+        end
+
+        GoogleSyncJob.perform_later
+        redirect_to contacts_path,
+                    notice: "Google re-sync has been queued for contacts in the recent window (creates and updates)."
+      end
+
       def show
         render "rails/contact/show"
       end
@@ -99,6 +130,12 @@ module Rails
         params.permit(:city, :region, :sync_eligible, :starred)
       end
 
+      def google_pending_sync_count
+        return 0 unless Rails::Contact.configuration.google_sync_enabled
+
+        Contact.where("google_resource_name IS NULL OR google_resource_name = ?", "").count
+      end
+
       def enqueue_index(contact_id)
         IndexContactJob.perform_later(contact_id)
       end

data/app/jobs/rails/contact/google_sync_unsynced_job.rb

@@ -0,0 +1,13 @@
+module Rails
+  module Contact
+    class GoogleSyncUnsyncedJob < ApplicationJob
+      queue_as :default
+
+      def perform
+        return unless Rails::Contact.configuration.google_sync_enabled
+
+        Google::SyncService.new.sync_unsynced!
+      end
+    end
+  end
+end

data/app/views/rails/contact/_google_sync_panel.html.erb

@@ -0,0 +1,27 @@
+<% if Rails::Contact.configuration.google_sync_enabled && Rails::Contact.configuration.google_sync_ui_on_index %>
+  <div class="rounded-xl border border-gray-200 bg-white p-4 shadow-sm space-y-4">
+    <h2 class="text-sm font-semibold text-gray-900">Google Contacts</h2>
+
+    <div>
+      <%= form_with url: google_sync_rolling_window_contacts_path, method: :post, local: true, class: "flex flex-wrap items-center gap-3" do %>
+        <%= submit_tag "Re-sync recent contacts with Google",
+              class: "inline-flex rounded-md bg-emerald-600 px-4 py-2 text-sm font-semibold text-white hover:bg-emerald-700",
+              data: { confirm: "Queue Google sync for the recent contact window? This updates contacts already in Google and creates any that are missing. Runs in the background." } %>
+      <% end %>
+      <p class="mt-2 text-xs text-gray-500">
+        Pushes the most recently updated contacts (up to your configured limit) to Google—new rows and changes to names, emails, phones, etc.
+      </p>
+    </div>
+
+    <% if @google_contacts_pending_sync.to_i.positive? %>
+      <div class="border-t border-gray-100 pt-4">
+        <%= form_with url: google_sync_unsynced_contacts_path, method: :post, local: true, class: "flex flex-wrap items-center gap-3" do %>
+          <%= submit_tag "Sync #{@google_contacts_pending_sync} contact#{'s' if @google_contacts_pending_sync != 1} with no Google link yet (any age)",
+                class: "inline-flex rounded-md border border-emerald-700 bg-white px-4 py-2 text-sm font-semibold text-emerald-800 hover:bg-emerald-50",
+                data: { confirm: "Queue Google sync for #{@google_contacts_pending_sync} contact(s) that are not linked yet? This runs in the background." } %>
+        <% end %>
+        <p class="mt-2 text-xs text-gray-500">For contacts outside the recent window or older imports that never received a Google <code class="rounded bg-gray-100 px-1">resourceName</code>.</p>
+      </div>
+    <% end %>
+  </div>
+<% end %>

data/app/views/rails/contact/_index_pagination.html.erb

@@ -0,0 +1,25 @@
+<% if defined?(@total_count) && @total_count.to_i > 0 %>
+  <div class="flex flex-col gap-3 border-t border-gray-200 bg-gray-50 px-4 py-3 sm:flex-row sm:items-center sm:justify-between">
+    <p class="text-sm text-gray-600">
+      <% start_i = (@page - 1) * @per_page + 1 %>
+      <% end_i = [ @page * @per_page, @total_count ].min %>
+      Showing <span class="font-medium text-gray-900"><%= start_i %>–<%= end_i %></span>
+      of <span class="font-medium text-gray-900"><%= number_with_delimiter(@total_count) %></span>
+      <% if @per_page.present? && @per_page > 0 %>
+        <span class="text-gray-500">(<%= @per_page %> per page)</span>
+      <% end %>
+    </p>
+    <% if defined?(@total_pages) && @total_pages > 1 %>
+      <div class="flex flex-wrap items-center gap-2">
+        <% qp = request.query_parameters.except(:page, :action, :controller) %>
+        <% if @page > 1 %>
+          <%= link_to "Previous", url_for(qp.merge(page: @page - 1)), class: "inline-flex rounded-md border border-gray-300 bg-white px-3 py-1.5 text-sm font-medium text-gray-700 hover:bg-gray-50" %>
+        <% end %>
+        <span class="text-sm text-gray-600">Page <%= @page %> / <%= @total_pages %></span>
+        <% if @page < @total_pages %>
+          <%= link_to "Next", url_for(qp.merge(page: @page + 1)), class: "inline-flex rounded-md border border-gray-300 bg-white px-3 py-1.5 text-sm font-medium text-gray-700 hover:bg-gray-50" %>
+        <% end %>
+      </div>
+    <% end %>
+  </div>
+<% end %>

data/app/views/rails/contact/index.html.erb

@@ -26,6 +26,8 @@
     <% end %>
   </div>
 
+  <%= render "rails/contact/google_sync_panel" %>
+
   <div class="rounded-xl border border-gray-200 bg-white p-4 shadow-sm">
     <%= form_with url: bulk_destroy_contacts_path, method: :post, local: true, class: "inline-flex items-center gap-2" do |f| %>
       <%= hidden_field_tag :ids, "", id: "bulk-ids-field" %>
@@ -76,6 +78,7 @@
         <% end %>
       </tbody>
     </table>
+    <%= render "rails/contact/index_pagination" %>
   </div>
 </div>
 

data/config/routes.rb

@@ -1,6 +1,8 @@
 Rails::Contact::Engine.routes.draw do
   post "/bulk_destroy", to: "contacts#bulk_destroy", as: :bulk_destroy_contacts
   post "/merge", to: "contacts#merge", as: :merge_contacts
+  post "/google_sync_unsynced", to: "contacts#google_sync_unsynced", as: :google_sync_unsynced_contacts
+  post "/google_sync_rolling_window", to: "contacts#google_sync_rolling_window", as: :google_sync_rolling_window_contacts
   get "/", to: "contacts#index", as: :contacts
   get "/new", to: "contacts#new", as: :new_contact
   post "/", to: "contacts#create"

data/lib/generators/rails/contact/install/templates/rails_contact.rb.tt

@@ -4,6 +4,8 @@ Rails::Contact.configure do |config|
   config.elasticsearch_url = ENV.fetch("ELASTICSEARCH_URL", "http://127.0.0.1:9200")
 
   config.google_sync_enabled = ENV.fetch("RAILS_CONTACT_GOOGLE_SYNC_ENABLED", "false") == "true"
+  # When true (default), contacts index renders the gem’s Google sync panel; set false to supply your own UI.
+  # config.google_sync_ui_on_index = false
   config.google_max_contacts = 25_000
   config.rolling_window_sort = :updated_at
 

data/lib/rails/contact/configuration.rb

@@ -2,7 +2,7 @@ module Rails
   module Contact
     class Configuration
       attr_accessor :contact_class_name, :elasticsearch_url, :search_backend,
-                    :google_sync_enabled, :google_max_contacts, :rolling_window_sort,
+                    :google_sync_enabled, :google_sync_ui_on_index, :google_max_contacts, :rolling_window_sort,
                     :google_client_id, :google_client_secret, :google_redirect_uri,
                     :google_token_path, :reset_index_on_boot, :default_per_page,
                     :inherit_host_layout,
@@ -13,6 +13,8 @@ module Rails
         @elasticsearch_url = ENV.fetch("ELASTICSEARCH_URL", "http://127.0.0.1:9200")
         @search_backend = :elasticsearch
         @google_sync_enabled = false
+        # When true with google_sync_enabled, the default index renders _google_sync_panel (host can override that partial).
+        @google_sync_ui_on_index = true
         @google_max_contacts = 25_000
         @rolling_window_sort = :updated_at
         @google_client_id = ENV["GOOGLE_CLIENT_ID"]

data/lib/rails/contact/google/sync_service.rb

@@ -13,6 +13,22 @@ module Rails
           end
         end
 
+        def unsynced_contacts_scope
+          Rails::Contact::Contact.where("google_resource_name IS NULL OR google_resource_name = ?", "")
+        end
+
+        def sync_unsynced!
+          unsynced_contacts_scope.in_batches(of: 100) do |batch|
+            batch.preload(:emails, :phones, :addresses).each do |contact|
+              sync_contact(contact)
+            rescue StandardError => e
+              Rails.logger.error(
+                "[Rails::Contact] Google sync_unsynced failed for contact #{contact.id}: #{e.class}: #{e.message}"
+              )
+            end
+          end
+        end
+
         def sync_contact(contact)
           payload = PayloadMapper.new(contact).to_people_payload
           response = if contact.google_resource_name.present?

data/lib/rails/contact/search/backends/database.rb

@@ -3,13 +3,19 @@ module Rails
     module Search
       module Backends
         class Database
-          def search(query, filters)
+          def search(query, filters, page:, per_page:)
+            offset = (page - 1) * per_page
             scope = Contact.includes(:emails, :phones, :labels).recent_first
             scope = apply_filters(scope, filters)
-            return scope.limit(limit) if query.blank?
+
+            if query.blank?
+              total = scope.count
+              records = scope.offset(offset).limit(per_page).to_a
+              return Search::Result.new(records: records, total_count: total, page: page, per_page: per_page)
+            end
 
             wildcard = "%#{query.downcase}%"
-            scope.left_joins(:emails, :phones, :labels).where(
+            filtered = scope.left_joins(:emails, :phones, :labels).where(
               "LOWER(rails_contact_contacts.given_name) LIKE :q OR "\
               "LOWER(rails_contact_contacts.family_name) LIKE :q OR "\
               "LOWER(COALESCE(rails_contact_contacts.metadata->>'company', '')) LIKE :q OR "\
@@ -19,7 +25,10 @@ module Rails
               "LOWER(rails_contact_labels.name) LIKE :q",
               q: wildcard,
               raw: "%#{query}%"
-            ).distinct.limit(limit)
+            ).distinct
+            total = filtered.count(:id)
+            records = filtered.offset(offset).limit(per_page).to_a
+            Search::Result.new(records: records, total_count: total, page: page, per_page: per_page)
           end
 
           private
@@ -34,10 +43,6 @@ module Rails
             end
             scoped
           end
-
-          def limit
-            Rails::Contact.configuration.default_per_page
-          end
         end
       end
     end

data/lib/rails/contact/search/backends/elasticsearch.rb

@@ -23,13 +23,16 @@ module Rails
             @client = client || self.class.default_client
           end
 
-          def search(query, filters)
-            response = @client.search(index: INDEX, body: search_body(query, filters))
-            ids = response.fetch("hits", {}).fetch("hits", []).map { |doc| doc["_id"] }
+          def search(query, filters, page:, per_page:)
+            response = @client.search(index: INDEX, body: search_body(query, filters, page: page, per_page: per_page))
+            hits = response.fetch("hits", {})
+            ids = hits.fetch("hits", []).map { |doc| doc["_id"] }
+            total_count = extract_total_hits(hits["total"])
             records_by_id = Contact.where(id: ids).index_by { |record| record.id.to_s }
-            ids.filter_map { |id| records_by_id[id.to_s] }
+            records = ids.filter_map { |id| records_by_id[id.to_s] }
+            Search::Result.new(records: records, total_count: total_count, page: page, per_page: per_page)
           rescue StandardError
-            Search::Backends::Database.new.search(query, filters)
+            Search::Backends::Database.new.search(query, filters, page: page, per_page: per_page)
           end
 
           def upsert(contact)
@@ -81,9 +84,12 @@ module Rails
             }
           end
 
-          def search_body(query, filters)
+          def search_body(query, filters, page:, per_page:)
+            from = (page - 1) * per_page
             {
-              size: Rails::Contact.configuration.default_per_page,
+              from: from,
+              size: per_page,
+              track_total_hits: true,
               sort: [ { updated_at: { order: "desc" } }, { id: { order: "desc" } } ],
               query: {
                 bool: {
@@ -94,6 +100,12 @@ module Rails
             }
           end
 
+          def extract_total_hits(raw)
+            return 0 if raw.nil?
+
+            raw.is_a?(Hash) ? raw.fetch("value", 0).to_i : raw.to_i
+          end
+
           def search_clause(query)
             return [ { match_all: {} } ] if query.blank?
 

data/lib/rails/contact/search/query.rb

@@ -2,13 +2,17 @@ module Rails
   module Contact
     module Search
       class Query
-        def initialize(query, filters: {})
+        MAX_PER_PAGE = 100
+
+        def initialize(query, filters: {}, page: nil, per_page: nil)
           @query = query
           @filters = filters.to_h.compact_blank
+          @page = page.to_i < 1 ? 1 : page.to_i
+          @per_page = resolve_per_page(per_page)
         end
 
         def call
-          backend.search(@query, @filters)
+          backend.search(@query, @filters, page: @page, per_page: @per_page)
         end
 
         private
@@ -21,6 +25,12 @@ module Rails
             Search::Backends::Database.new
           end
         end
+
+        def resolve_per_page(per_page)
+          p = per_page.to_i
+          p = Rails::Contact.configuration.default_per_page if p <= 0
+          [ p, MAX_PER_PAGE ].min
+        end
       end
     end
   end

data/lib/rails/contact/search/result.rb

@@ -0,0 +1,13 @@
+module Rails
+  module Contact
+    module Search
+      Result = Struct.new(:records, :total_count, :page, :per_page, keyword_init: true) do
+        def total_pages
+          return 0 if total_count.zero? || per_page.zero?
+
+          (total_count.to_f / per_page).ceil
+        end
+      end
+    end
+  end
+end

data/lib/rails/contact/version.rb

@@ -1,5 +1,5 @@
 module Rails
   module Contact
-    VERSION = "0.1.7"
+    VERSION = "0.1.11"
   end
 end

data/lib/rails/contact.rb

@@ -4,6 +4,7 @@ require "rails/contact/engine"
 require "rails/contact/configuration"
 require "rails/contact/orm"
 require "rails/contact/routing"
+require "rails/contact/search/result"
 require "rails/contact/search/query"
 require "rails/contact/search/backends/database"
 require "rails/contact/search/backends/elasticsearch"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-contact
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.11
 platform: ruby
 authors:
 - Kshitiz Sinha
@@ -119,6 +119,7 @@ files:
 - app/helpers/rails/contact/application_helper.rb
 - app/jobs/rails/contact/application_job.rb
 - app/jobs/rails/contact/google_sync_job.rb
+- app/jobs/rails/contact/google_sync_unsynced_job.rb
 - app/jobs/rails/contact/index_contact_job.rb
 - app/mailers/rails/contact/application_mailer.rb
 - app/mailers/rails/contact/contact_mailer.rb
@@ -137,6 +138,8 @@ files:
 - app/views/rails/contact/_email_fields.html.erb
 - app/views/rails/contact/_event_fields.html.erb
 - app/views/rails/contact/_form.html.erb
+- app/views/rails/contact/_google_sync_panel.html.erb
+- app/views/rails/contact/_index_pagination.html.erb
 - app/views/rails/contact/_nested_fields_script.html.erb
 - app/views/rails/contact/_phone_fields.html.erb
 - app/views/rails/contact/_stylesheet.html.erb
@@ -181,6 +184,7 @@ files:
 - lib/rails/contact/search/backends/database.rb
 - lib/rails/contact/search/backends/elasticsearch.rb
 - lib/rails/contact/search/query.rb
+- lib/rails/contact/search/result.rb
 - lib/rails/contact/version.rb
 - lib/tasks/rails/contact_tasks.rake
 - spec/concerns/filterable_spec.rb