Fix assistant creation

app/controllers/base_assistants_controller.rb

@@ -8,7 +8,7 @@ def index
 
   # POST /assistants or /assistants.json
   def create
-    json = JSON.parse(params[:json])
+    json = JSON.parse(params[:json]) if params[:json].present?
     @assistant = Assistant.new(json || assistant_params)
 
     authorize @assistant
@@ -58,6 +58,6 @@ def set_assistant
 
   # Only allow a list of trusted parameters through.
   def assistant_params
-    params.require(:assistant).permit(:libraries, :name, :input, :output, :context, :instructions, :description, :status, :quip_url, :confluence_spaces)
+    params.require(:assistant).permit(:libraries, :name, :input, :output, :context, :instructions, :description, :status, :quip_url, :confluence_spaces, :user_id)
   end
 end

app/models/assistant.rb

@@ -8,8 +8,6 @@ class Assistant < ApplicationRecord
   validates :libraries, presence: true
 
   validates :input, presence: true
-  validates :instructions, presence: true
-  validates :output, presence: true
 
   validate :libraries_must_be_csv_with_numbers
 

app/views/assistants/_form.html.erb

@@ -53,10 +53,6 @@
     <%= form.label :context %>
     <%= form.text_area :context, rows: 4, class: "block shadow rounded-md border border-gray-400 outline-none px-3 py-2 mt-2 w-full" %>
   </div>
-  <div class="my-5">
-    <%= label_tag :json %>
-    <%= text_area_tag :json, nil, rows: 4, class: "block shadow rounded-md border border-gray-400 outline-none px-3 py-2 mt-2 w-full" %>
-  </div>
   <div class="inline">
     <%= form.submit "Save", class: "rounded-lg py-3 px-5 bg-sky-600 text-white inline-block font-medium cursor-pointer" %>
   </div>

app/views/assistants/index.html.erb

@@ -1,12 +1,16 @@
 <div class="w-full">
   <% content_for :title, "Assistants" %>
   <div class="flex justify-between items-center">
-    <%= render 'shared/page_header', title: 'Assistants', subtitle: 'Assistants have special instructions and context to answer specific problems better than general Q/A.  Click "Chat" to begin!' %>
-    <% if policy(Assistant.new).edit? %>
-      <%= link_to "New +", new_assistant_path, class: "rounded-lg py-3 px-5 bg-sky-900 text-white block font-medium" %>
-      <%= link_to "Import", import_assistants_path, class: "rounded-lg py-3 px-5 bg-sky-900 text-white block font-medium" %>
-    <% end %>
-  </div>
+    <%= render 'shared/page_header', title: 'Assistants', subtitle: 'Assistants have special instructions and context to answer specific problems better than general Q/A. Click "Chat" to begin!' %>
+    <div>
+      <% if policy(Assistant.new).edit? %>
+        <div class="inline-flex rounded-md shadow-sm" role="group">
+          <%= link_to "New +", new_assistant_path, class: "border border-sky-600 border-r-0 py-3 px-5 bg-sky-100 text-sky-600 font-medium border border-sky-900 rounded-l-lg hover:bg-sky-700 hover:text-sky-100" %>
+          <%= link_to "Import", import_assistants_path, class: "border border-sky-600 py-3 px-5 bg-sky-100 text-sky-600 font-medium border border-sky-900 rounded-r-lg hover:bg-sky-700 hover:text-sky-100" %>
+        </div>
+      <% end %>
+    </div>
+  </div> 
   <div id="assistants" class="mt-5 space-y-4">
     <% @assistants.each do |assistant| %>
       <div class="flex justify-between items-center bg-white rounded-lg shadow-lg p-4 border-stone-200 border">

spec/models/assistant_spec.rb

@@ -6,8 +6,6 @@
   describe 'validations' do
     it { should validate_presence_of(:libraries) }
     it { should validate_presence_of(:input) }
-    it { should validate_presence_of(:instructions) }
-    it { should validate_presence_of(:output) }
 
     context 'libraries CSV validation' do
       it 'is valid with a valid CSV string of numbers' do