diff options
Diffstat (limited to 'app/controllers/projects_controller.rb')
-rw-r--r-- | app/controllers/projects_controller.rb | 40 |
1 files changed, 19 insertions, 21 deletions
diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb index ebffb62cff3..bc48ebd1c74 100644 --- a/app/controllers/projects_controller.rb +++ b/app/controllers/projects_controller.rb @@ -31,10 +31,6 @@ class ProjectsController < Projects::ApplicationController # Project Export Rate Limit before_action :export_rate_limit, only: [:export, :download_export, :generate_new_export] - before_action do - push_frontend_feature_flag(:vue_notification_dropdown, @project, default_enabled: :yaml) - end - before_action only: [:edit] do push_frontend_feature_flag(:allow_editing_commit_messages, @project) end @@ -74,7 +70,11 @@ class ProjectsController < Projects::ApplicationController @project = ::Projects::CreateService.new(current_user, project_params(attributes: project_params_create_attributes)).execute if @project.saved? - experiment(:new_project_readme, actor: current_user).track(:created, property: active_new_project_tab) + experiment(:new_project_readme, actor: current_user).track( + :created, + property: active_new_project_tab, + value: project_params[:initialize_with_readme].to_i + ) redirect_to( project_path(@project, custom_import_params), notice: _("Project '%{project_name}' was successfully created.") % { project_name: @project.name } @@ -90,21 +90,13 @@ class ProjectsController < Projects::ApplicationController # Refresh the repo in case anything changed @repository = @project.repository - respond_to do |format| - if result[:status] == :success - flash[:notice] = _("Project '%{project_name}' was successfully updated.") % { project_name: @project.name } - - format.html do - redirect_to(edit_project_path(@project, anchor: 'js-general-project-settings')) - end - else - flash[:alert] = result[:message] - @project.reset - - format.html { render_edit } - end - - format.js + if result[:status] == :success + flash[:notice] = _("Project '%{project_name}' was successfully updated.") % { project_name: @project.name } + redirect_to(edit_project_path(@project, anchor: 'js-general-project-settings')) + else + flash[:alert] = result[:message] + @project.reset + render 'edit' end end @@ -329,6 +321,11 @@ class ProjectsController < Projects::ApplicationController if can?(current_user, :download_code, @project) return render 'projects/no_repo' unless @project.repository_exists? + if @project.can_current_user_push_to_default_branch? + property = @project.empty_repo? ? 'empty' : 'nonempty' + experiment(:empty_repo_upload, project: @project).track(:view_project_show, property: property) + end + if @project.empty_repo? record_experiment_user(:invite_members_empty_project_version_a) @@ -393,6 +390,7 @@ class ProjectsController < Projects::ApplicationController metrics_dashboard_access_level analytics_access_level operations_access_level + security_and_compliance_access_level ] end @@ -523,7 +521,7 @@ class ProjectsController < Projects::ApplicationController def export_rate_limit prefixed_action = "project_#{params[:action]}".to_sym - project_scope = params[:action] == :download_export ? @project : nil + project_scope = params[:action] == 'download_export' ? @project : nil if rate_limiter.throttled?(prefixed_action, scope: [current_user, project_scope].compact) rate_limiter.log_request(request, "#{prefixed_action}_request_limit".to_sym, current_user) |