Removed API route for v1/search, only using v2/search now
• Removed: - API route for v1/search, only using v2/search now - v2/search requires user (for now)
This commit is contained in:
parent
e754c6219b
commit
b15c905143
|
@ -1,27 +0,0 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
class Api::V1::SearchController < Api::BaseController
|
||||
before_action :require_user!
|
||||
|
||||
RESULTS_LIMIT = 25
|
||||
|
||||
def index
|
||||
@search = Search.new(search_results)
|
||||
render json: @search, serializer: REST::SearchSerializer
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def search_results
|
||||
SearchService.new.call(
|
||||
params[:q],
|
||||
current_account,
|
||||
limit_param(RESULTS_LIMIT),
|
||||
search_params.merge(resolve: truthy_param?(:resolve))
|
||||
)
|
||||
end
|
||||
|
||||
def search_params
|
||||
params.permit(:type, :onlyVerified, :offset, :min_id, :max_id, :account_id)
|
||||
end
|
||||
end
|
|
@ -1,8 +1,29 @@
|
|||
# frozen_string_literal: true
|
||||
|
||||
class Api::V2::SearchController < Api::V1::SearchController
|
||||
class Api::V2::SearchController < Api::BaseController
|
||||
before_action :require_user!
|
||||
|
||||
RESULTS_LIMIT = 25
|
||||
|
||||
def index
|
||||
@search = Search.new(search_results)
|
||||
render json: @search, serializer: REST::V2::SearchSerializer
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def search_results
|
||||
SearchService.new.call(
|
||||
params[:q],
|
||||
current_account,
|
||||
limit_param(RESULTS_LIMIT),
|
||||
search_params.merge(resolve: truthy_param?(:resolve))
|
||||
)
|
||||
end
|
||||
|
||||
def search_params
|
||||
params.permit(:type, :onlyVerified, :offset, :min_id, :max_id, :account_id)
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
|
|
|
@ -294,7 +294,6 @@ Rails.application.routes.draw do
|
|||
end
|
||||
end
|
||||
|
||||
get '/search', to: 'search#index', as: :search
|
||||
get '/account_by_username/:username', to: 'account_by_username#show', username: username_regex
|
||||
|
||||
namespace :apps do
|
||||
|
|
Loading…
Reference in New Issue