Refactor stock search and add country code filter option

maybe-finance · Nov 20, 2024 · c8fb28a · c8fb28a
1 parent dc3f476
commit c8fb28a
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/app/controllers/stocks_controller.rb b/app/controllers/stocks_controller.rb
@@ -33,7 +33,9 @@ def index
     end
 
     if params[:combobox].present?
-      scope = Stock.order(:name).search(params[:q])
+      scope = Stock.order(:name)
+      scope = scope.where(country_code: params[:country_code]) if params[:country_code].present?
+      scope = scope.search(params[:q])
       @pagy, @stocks = pagy(scope, limit: 27, size: [ 1, 3, 3, 1 ])
       @total_stocks = @pagy.count
       render :index, variants: [ :combobox ]

diff --git a/app/presenters/tool/presenter/compound_interest_calculator.rb b/app/presenters/tool/presenter/compound_interest_calculator.rb
@@ -4,6 +4,7 @@ class Tool::Presenter::CompoundInterestCalculator < Tool::Presenter
   attribute :initial_investment, :tool_float, default: 0.0
   attribute :monthly_contribution, :tool_float, default: 0.0
   attribute :years_to_grow, :tool_float, default: 0.0, min: 0.0, max: 150.0
+  attribute :filter, :string
 
   def blank?
     [ initial_investment, monthly_contribution, years_to_grow, annual_interest_rate ].all?(&:zero?)

diff --git a/app/views/tools/widgets/forms/_inside_trading_tracker.html.erb b/app/views/tools/widgets/forms/_inside_trading_tracker.html.erb
@@ -1,6 +1,6 @@
 <%= form_with method: :get, data: { controller: "insider-form" } do |form| %>
   <div class="search-combobox">
-    <%= combobox_tag "symbol", stocks_path(combobox: true),
+    <%= combobox_tag "symbol", stocks_path(combobox: true, country_code: "US"),
       label: "Company Symbol",
       placeholder: "Search for a stock",
       required: true,