content_signals 0.1.11 → 0.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79d109a0ff6e808dd29af442535c5041b0870b82057c9b9bad365c995d1267b2
-  data.tar.gz: 130730c5ad2c79beec04216b1d939cd359945e3af5c6c389e55f37c87b0d9ec7
+  metadata.gz: e2c1d01c7f6f684b573250d04429a365fa3ab703cb8dddac8ddac0a7131580b8
+  data.tar.gz: bb343e43a050d8007817062d8de52f64f3ed54ee51d459b1a4988689e420083d
 SHA512:
-  metadata.gz: fd2c5897912d51787f2a2f9b4cc025d15a29782253e05bb12f1d5f35376e1392a786b4caf81563752d4f6ac05a309b28788673f6d10da88df94df411ffacde61
-  data.tar.gz: 87a2efb0694d43192fefef083f8e12fe815a54859a463833213c1d091b00ca1b7b2584d2576d0dae907ca384ec8785639f6a59d17182bcf749757f6dadb50758
+  metadata.gz: 4ca85cd8372d706ff93c3c9faa11ecf2920eff399fd7fa276f7f6cd09d7e81b569a8977cb1998189c43790c510500b8530f00a88a2d9860819909bc656eaa85e
+  data.tar.gz: 86dd62f5920ddc4565f4a619ba34ff9c57ef130e12f3a08af719fa11e6895bb28cd869d3278f338f3ef7004e231122b97cf9f1fe399f0d737d771cea3bff21c2

data/lib/content_signals/models/page_view.rb

@@ -6,33 +6,21 @@ module ContentSignals
 
     # Associations
     belongs_to :trackable, polymorphic: true, counter_cache: :page_views_count
-    belongs_to :tenant, optional: true if ContentSignals.configuration.multitenancy?
     belongs_to :user, optional: true
 
+    # tenant_id is a plain string (UUID), not a foreign key — declare explicitly
+    # Guard allows db:migrate to run on a fresh DB before the table exists
+    attribute :tenant_id, :string if connection.table_exists?(table_name) rescue nil
+
     # Validations
     validates :trackable_type, :trackable_id, :visitor_id, :viewed_at, presence: true
     validates :country_code, length: { maximum: 2 }, allow_nil: true
     validates :device_type, inclusion: { in: %w[desktop mobile tablet] }, allow_nil: true
     validates :app_platform, inclusion: { in: %w[hybrid native] }, allow_nil: true
 
-    # Default scope for tenant isolation (when multitenancy is enabled)
-    if ContentSignals.configuration.multitenancy?
-      default_scope -> { where(tenant_id: current_tenant_id) }
-
-      def self.current_tenant_id
-        return nil unless ContentSignals.configuration.multitenancy?
-
-        method_name = ContentSignals.configuration.current_tenant_method
-        return nil unless method_name
-
-        # Try to get tenant_id from Current, controller, or thread
-        if defined?(Current) && Current.respond_to?(method_name)
-          Current.send(method_name)
-        elsif Thread.current[method_name]
-          Thread.current[method_name]
-        end
-      end
-    end
+    # Tenant filtering — use explicitly instead of default_scope to avoid
+    # silent NULL filtering in background jobs, console, and migrations.
+    scope :for_tenant, ->(id) { where(tenant_id: id) }
 
     # Time period scopes
     scope :today, -> { where("viewed_at >= ?", Time.current.beginning_of_day) }

data/lib/content_signals/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ContentSignals
-  VERSION = "0.1.11"
+  VERSION = "0.1.15"
 end

data/lib/generators/content_signals/templates/create_content_signals_page_views.rb.erb

@@ -1,8 +1,8 @@
 class CreateContentSignalsPageViews < ActiveRecord::Migration<%= migration_version %>
   def change
     create_table :content_signals_page_views do |t|
-      # Multi-tenancy support (optional)
-      t.integer :tenant_id, index: true
+      # Multi-tenancy support (optional, string to support both UUID and integer tenant IDs)
+      t.string :tenant_id, index: true
 
       # Polymorphic trackable (Page, Profile, Event, etc.)
       t.string :trackable_type, null: false

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: content_signals
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.15
 platform: ruby
 authors:
 - Ionuț Munteanu Alexandru