Merge pull request #50 from ThinkThinkAI/apikey_notrequired

apikey is technically not required

Author: rootedbox

VERSION

@@ -1 +1 @@
-1.0.23-alpha
+1.0.24-alpha

app/models/data_source.rb

@@ -10,6 +10,9 @@ class DataSource < ApplicationRecord
   before_save :encrypt_password, if: :password_changed?
   before_save :unset_other_connected_sources, if: :connected
 
+  # has_many :data_source_users
+  # has_many :users, through: :data_source_users
+
   after_save :build_tables_if_connected
 
   after_destroy :activate_first_available_data_source_if_none_active

app/models/user.rb

@@ -23,7 +23,7 @@ def self.from_omniauth(auth)
   end
 
   def settings_incomplete?
-    ai_url.blank? || ai_model.blank? || ai_api_key.blank?
+    ai_url.blank? || ai_model.blank?
   end
 
   private

app/services/ai_service.rb

@@ -30,7 +30,7 @@ def query_system_message(data_source)
       You will be asked by the user for information or to perform a task on the database.
       Your response will only be in JSON which will include up too 2 things.
       1. the query to do what was requested by the user.
-      2. if the query is a delete or update then a select query to preview which records would be affected.
+      2. if the query is not a select query we will need to preview which records would be affected. examples are drop, update, delete.
       The JSON will look like this. { sql: "query needed", preview: "preview query" }
       Do not include the preview if it is not necesary.
       Below is the schema for the database.

spec/models/user_spec.rb

@@ -87,18 +87,6 @@
         expect(user.settings_incomplete?).to be true
       end
     end
-
-    context 'when ai_api_key is blank' do
-      before do
-        user.ai_url = 'http://example.com'
-        user.ai_model = 'example_model'
-        user.ai_api_key = ''
-      end
-
-      it 'returns true' do
-        expect(user.settings_incomplete?).to be true
-      end
-    end
   end
 
   describe '.from_omniauth' do