resolve conversations from PR

Gemfile

@@ -57,7 +57,7 @@ group :development, :test do
 end
 
 group :development do
-  gem 'bullet', '~> 7.1', '>= 7.1.6'
+  gem 'bullet'
   gem 'listen', '~> 3.0.5'
   gem 'rb-readline'
   # Spring speeds up development by keeping your application
@@ -71,6 +71,7 @@ group :test do
   gem 'capybara'
   gem 'selenium-webdriver'
   gem 'simplecov'
+  # Use fixed version of webdrivers to avoid compatibility issues with chrome and chromedriver
   gem 'webdrivers', '~> 5.3', '>= 5.3.1'
 end
 # Windows does not include zoneinfo files, so bundle the tzinfo-data gem

Gemfile.lock

@@ -435,7 +435,7 @@ DEPENDENCIES
   bleib (= 0.0.8)
   bootsnap
   brakeman
-  bullet (~> 7.1, >= 7.1.6)
+  bullet
   byebug
   capybara
   carrierwave

app/controllers/skills_controller.rb

@@ -10,6 +10,12 @@ class SkillsController < CrudController
 
   self.permitted_attrs = %i[title radar portfolio default_set category_id]
 
+  def index
+    super
+    require 'pry'
+    binding.pry
+  end
+
   def create
     super(:location => skills_path,
           render_on_failure: 'skills/form_update',

app/javascript/controllers/dropdown_controller.js

@@ -2,8 +2,7 @@ import { Controller } from "@hotwired/stimulus"
 
 // Connects to data-controller="dropdown-links"
 export default class extends Controller {
-  connect() {
-  }
+  connect() {}
 
     handleChange(event) {
         window.location.href = event.target.dataset.value + event.target.value;

app/views/people/_search.html.haml

@@ -1,3 +1,3 @@
 = form_with do |f|
   %section{"data-controller"=>"dropdown"}
-    = f.collection_select :person_id, Person.all.sort_by(&:name), :id, :name, {:selected => person.id} , {class: "form-select w-100", "data-action": "change->dropdown#handleChange", "data-value": "/people/"}
+    = f.collection_select :person_id, Person.all.sort_by(&:name), :id, :name, {:selected => person&.id} , {class: "form-select w-100", "data-action": "change->dropdown#handleChange", "data-value": "/people/"}