Add render_projects method to controller
- Render EE specific partial if exists
This commit is contained in:
parent
54cc3b6492
commit
dab00e6929
|
@ -6,18 +6,14 @@ class Dashboard::ProjectsController < Dashboard::ApplicationController
|
|||
|
||||
prepend_before_action(only: [:index]) { authenticate_sessionless_user!(:rss) }
|
||||
before_action :set_non_archived_param
|
||||
before_action :projects, only: [:index]
|
||||
before_action :default_sorting
|
||||
skip_cross_project_access_check :index, :starred
|
||||
|
||||
def index
|
||||
@projects = load_projects(params.merge(non_public: true))
|
||||
|
||||
respond_to do |format|
|
||||
format.html do
|
||||
# n+1: https://gitlab.com/gitlab-org/gitlab-ce/issues/40260
|
||||
Gitlab::GitalyClient.allow_n_plus_1_calls do
|
||||
render
|
||||
end
|
||||
render_projects
|
||||
end
|
||||
format.atom do
|
||||
load_events
|
||||
|
@ -51,6 +47,17 @@ class Dashboard::ProjectsController < Dashboard::ApplicationController
|
|||
|
||||
private
|
||||
|
||||
def projects
|
||||
@projects ||= load_projects(params.merge(non_public: true))
|
||||
end
|
||||
|
||||
def render_projects
|
||||
# n+1: https://gitlab.com/gitlab-org/gitlab-ce/issues/40260
|
||||
Gitlab::GitalyClient.allow_n_plus_1_calls do
|
||||
render
|
||||
end
|
||||
end
|
||||
|
||||
def default_sorting
|
||||
params[:sort] ||= 'latest_activity_desc'
|
||||
@sort = params[:sort]
|
||||
|
|
|
@ -241,6 +241,7 @@ module ProjectsHelper
|
|||
|
||||
# TODO: Remove this method when removing the feature flag
|
||||
# https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/11209#note_162234863
|
||||
# make sure to remove from the EE specific controller as well: ee/app/controllers/ee/dashboard/projects_controller.rb
|
||||
def show_projects?(projects, params)
|
||||
Feature.enabled?(:project_list_filter_bar) || !!(params[:personal] || params[:name] || any_projects?(projects))
|
||||
end
|
||||
|
|
|
@ -10,4 +10,6 @@
|
|||
= render 'layouts/page', sidebar: sidebar, nav: nav
|
||||
= footer_message
|
||||
|
||||
= render_if_exists "shared/onboarding_guide"
|
||||
|
||||
= yield :scripts_body
|
||||
|
|
Loading…
Reference in New Issue