Commit aa2c729e authored by Kamil Trzciński's avatar Kamil Trzciński

Merge branch '41766-create-releases-page' into 'master'

Creates index page for releases behind a feature flag

See merge request gitlab-org/gitlab-ce!23687
parents 257a61e8 867a1acc
...@@ -8,6 +8,7 @@ export default class ShortcutsNavigation extends Shortcuts { ...@@ -8,6 +8,7 @@ export default class ShortcutsNavigation extends Shortcuts {
Mousetrap.bind('g p', () => findAndFollowLink('.shortcuts-project')); Mousetrap.bind('g p', () => findAndFollowLink('.shortcuts-project'));
Mousetrap.bind('g v', () => findAndFollowLink('.shortcuts-project-activity')); Mousetrap.bind('g v', () => findAndFollowLink('.shortcuts-project-activity'));
Mousetrap.bind('g r', () => findAndFollowLink('.shortcuts-project-releases'));
Mousetrap.bind('g f', () => findAndFollowLink('.shortcuts-tree')); Mousetrap.bind('g f', () => findAndFollowLink('.shortcuts-tree'));
Mousetrap.bind('g c', () => findAndFollowLink('.shortcuts-commits')); Mousetrap.bind('g c', () => findAndFollowLink('.shortcuts-commits'));
Mousetrap.bind('g j', () => findAndFollowLink('.shortcuts-builds')); Mousetrap.bind('g j', () => findAndFollowLink('.shortcuts-builds'));
......
...@@ -4,39 +4,16 @@ class Projects::ReleasesController < Projects::ApplicationController ...@@ -4,39 +4,16 @@ class Projects::ReleasesController < Projects::ApplicationController
# Authorize # Authorize
before_action :require_non_empty_project before_action :require_non_empty_project
before_action :authorize_download_code! before_action :authorize_download_code!
before_action :authorize_push_code! before_action :check_releases_page_feature_flag
before_action :tag
before_action :release
def edit def index
end
def update
# Release belongs to Tag which is not active record object,
# it exists only to save a description to each Tag.
# If description is empty we should destroy the existing record.
if release_params[:description].present?
release.update(release_params)
else
release.destroy
end
redirect_to project_tag_path(@project, @tag.name)
end end
private private
def tag def check_releases_page_feature_flag
@tag ||= @repository.find_tag(params[:tag_id]) return render_404 unless Feature.enabled?(:releases_page)
end
# rubocop: disable CodeReuse/ActiveRecord
def release
@release ||= @project.releases.find_or_initialize_by(tag: @tag.name)
end
# rubocop: enable CodeReuse/ActiveRecord
def release_params push_frontend_feature_flag(:releases_page)
params.require(:release).permit(:description)
end end
end end
# frozen_string_literal: true
class Projects::Tags::ReleasesController < Projects::ApplicationController
# Authorize
before_action :require_non_empty_project
before_action :authorize_download_code!
before_action :authorize_push_code!
before_action :tag
before_action :release
def edit
end
def update
# Release belongs to Tag which is not active record object,
# it exists only to save a description to each Tag.
# If description is empty we should destroy the existing record.
if release_params[:description].present?
release.update(release_params)
else
release.destroy
end
redirect_to project_tag_path(@project, @tag.name)
end
private
def tag
@tag ||= @repository.find_tag(params[:tag_id])
end
# rubocop: disable CodeReuse/ActiveRecord
def release
@release ||= @project.releases.find_or_initialize_by(tag: @tag.name)
end
# rubocop: enable CodeReuse/ActiveRecord
def release_params
params.require(:release).permit(:description)
end
end
...@@ -42,7 +42,7 @@ class Projects::TagsController < Projects::ApplicationController ...@@ -42,7 +42,7 @@ class Projects::TagsController < Projects::ApplicationController
# rubocop: enable CodeReuse/ActiveRecord # rubocop: enable CodeReuse/ActiveRecord
def create def create
result = Tags::CreateService.new(@project, current_user) result = ::Tags::CreateService.new(@project, current_user)
.execute(params[:tag_name], params[:ref], params[:message], params[:release_description]) .execute(params[:tag_name], params[:ref], params[:message], params[:release_description])
if result[:status] == :success if result[:status] == :success
...@@ -58,7 +58,7 @@ class Projects::TagsController < Projects::ApplicationController ...@@ -58,7 +58,7 @@ class Projects::TagsController < Projects::ApplicationController
end end
def destroy def destroy
result = Tags::DestroyService.new(project, current_user).execute(params[:id]) result = ::Tags::DestroyService.new(project, current_user).execute(params[:id])
respond_to do |format| respond_to do |format|
if result[:status] == :success if result[:status] == :success
......
...@@ -277,7 +277,7 @@ module ProjectsHelper ...@@ -277,7 +277,7 @@ module ProjectsHelper
nav_tabs = [:home] nav_tabs = [:home]
if !project.empty_repo? && can?(current_user, :download_code, project) if !project.empty_repo? && can?(current_user, :download_code, project)
nav_tabs << [:files, :commits, :network, :graphs, :forks] nav_tabs << [:files, :commits, :network, :graphs, :forks, :releases]
end end
if project.repo_exists? && can?(current_user, :read_merge_request, project) if project.repo_exists? && can?(current_user, :read_merge_request, project)
...@@ -533,6 +533,7 @@ module ProjectsHelper ...@@ -533,6 +533,7 @@ module ProjectsHelper
%w[ %w[
projects#show projects#show
projects#activity projects#activity
releases#index
cycle_analytics#show cycle_analytics#show
] ]
end end
...@@ -564,7 +565,6 @@ module ProjectsHelper ...@@ -564,7 +565,6 @@ module ProjectsHelper
projects/repositories projects/repositories
tags tags
branches branches
releases
graphs graphs
network network
] ]
......
...@@ -29,6 +29,11 @@ ...@@ -29,6 +29,11 @@
= link_to activity_project_path(@project), title: _('Activity'), class: 'shortcuts-project-activity' do = link_to activity_project_path(@project), title: _('Activity'), class: 'shortcuts-project-activity' do
%span= _('Activity') %span= _('Activity')
- if project_nav_tab?(:releases) && Feature.enabled?(:releases_page)
= nav_link(controller: :releases) do
= link_to project_releases_path(@project), title: _('Releases'), class: 'shortcuts-project-releases' do
%span= _('Releases')
= render_if_exists 'projects/sidebar/security_dashboard' = render_if_exists 'projects/sidebar/security_dashboard'
- if can?(current_user, :read_cycle_analytics, @project) - if can?(current_user, :read_cycle_analytics, @project)
...@@ -62,7 +67,7 @@ ...@@ -62,7 +67,7 @@
= link_to project_branches_path(@project) do = link_to project_branches_path(@project) do
= _('Branches') = _('Branches')
= nav_link(controller: [:tags, :releases]) do = nav_link(controller: [:tags]) do
= link_to project_tags_path(@project) do = link_to project_tags_path(@project) do
= _('Tags') = _('Tags')
......
- @no_container = true
- page_title _('Releases')
%div{ 'class' => container_class }
#js-releases-page
...@@ -95,6 +95,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do ...@@ -95,6 +95,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
end end
end end
resources :releases, only: [:index]
resources :forks, only: [:index, :new, :create] resources :forks, only: [:index, :new, :create]
resource :import, only: [:new, :create, :show] resource :import, only: [:new, :create, :show]
......
...@@ -55,7 +55,7 @@ scope format: false do ...@@ -55,7 +55,7 @@ scope format: false do
resources :branches, only: [:index, :new, :create, :destroy] resources :branches, only: [:index, :new, :create, :destroy]
delete :merged_branches, controller: 'branches', action: :destroy_all_merged delete :merged_branches, controller: 'branches', action: :destroy_all_merged
resources :tags, only: [:index, :show, :new, :create, :destroy] do resources :tags, only: [:index, :show, :new, :create, :destroy] do
resource :release, only: [:edit, :update] resource :release, controller: 'tags/releases', only: [:edit, :update]
end end
resources :protected_branches, only: [:index, :show, :create, :update, :destroy] resources :protected_branches, only: [:index, :show, :create, :update, :destroy]
......
...@@ -5506,6 +5506,9 @@ msgstr "" ...@@ -5506,6 +5506,9 @@ msgstr ""
msgid "Related merge requests" msgid "Related merge requests"
msgstr "" msgstr ""
msgid "Releases"
msgstr ""
msgid "Remind later" msgid "Remind later"
msgstr "" msgstr ""
......
# frozen_string_literal: true
require 'spec_helper' require 'spec_helper'
describe Projects::ReleasesController do describe Projects::ReleasesController do
let!(:project) { create(:project, :repository) } let!(:project) { create(:project, :repository, :public) }
let!(:user) { create(:user) } let!(:user) { create(:user) }
let!(:release) { create(:release, project: project) }
let!(:tag) { release.tag }
before do before do
project.add_developer(user) stub_feature_flags(releases_page: true)
sign_in(user)
end end
describe 'GET #edit' do describe 'GET #index' do
it 'initializes a new release' do it 'renders a 200' do
tag_id = release.tag get_index
project.releases.destroy_all # rubocop: disable DestroyAll
get :edit, namespace_id: project.namespace, project_id: project, tag_id: tag_id expect(response.status).to eq(200)
release = assigns(:release)
expect(release).not_to be_nil
expect(release).not_to be_persisted
end end
it 'retrieves an existing release' do context 'when the project is private' do
get :edit, namespace_id: project.namespace, project_id: project, tag_id: release.tag let!(:project) { create(:project, :repository, :private) }
it 'renders a 302' do
get_index
release = assigns(:release) expect(response.status).to eq(302)
expect(release).not_to be_nil
expect(release).to be_persisted
end end
it 'renders a 200 for a logged in developer' do
project.add_developer(user)
sign_in(user)
get_index
expect(response.status).to eq(200)
end end
describe 'PUT #update' do it 'renders a 404 when logged in but not in the project' do
it 'updates release note description' do sign_in(user)
update_release('description updated')
get_index
release = project.releases.find_by_tag(tag) expect(response.status).to eq(404)
expect(release.description).to eq("description updated") end
end end
it 'deletes release note when description is null' do context 'when releases_page feature flag is disabled' do
expect { update_release('') }.to change(project.releases, :count).by(-1) before do
stub_feature_flags(releases_page: false)
end
it 'renders a 404' do
get_index
expect(response.status).to eq(404)
end end
end end
end
private
def update_release(description) def get_index
put :update, get :index, namespace_id: project.namespace, project_id: project
namespace_id: project.namespace.to_param,
project_id: project,
tag_id: release.tag,
release: { description: description }
end end
end end
# frozen_string_literal: true
require 'spec_helper'
describe Projects::Tags::ReleasesController do
let!(:project) { create(:project, :repository) }
let!(:user) { create(:user) }
let!(:release) { create(:release, project: project) }
let!(:tag) { release.tag }
before do
project.add_developer(user)
sign_in(user)
end
describe 'GET #edit' do
it 'initializes a new release' do
tag_id = release.tag
project.releases.destroy_all # rubocop: disable DestroyAll
get :edit, namespace_id: project.namespace, project_id: project, tag_id: tag_id
release = assigns(:release)
expect(release).not_to be_nil
expect(release).not_to be_persisted
end
it 'retrieves an existing release' do
get :edit, namespace_id: project.namespace, project_id: project, tag_id: release.tag
release = assigns(:release)
expect(release).not_to be_nil
expect(release).to be_persisted
end
end
describe 'PUT #update' do
it 'updates release note description' do
update_release('description updated')
release = project.releases.find_by_tag(tag)
expect(release.description).to eq("description updated")
end
it 'deletes release note when description is null' do
expect { update_release('') }.to change(project.releases, :count).by(-1)
end
end
def update_release(description)
put :update,
namespace_id: project.namespace.to_param,
project_id: project,
tag_id: release.tag,
release: { description: description }
end
end
...@@ -49,4 +49,30 @@ describe 'layouts/nav/sidebar/_project' do ...@@ -49,4 +49,30 @@ describe 'layouts/nav/sidebar/_project' do
expect(rendered).to have_css('.sidebar-top-level-items > li.active', text: 'Registry') expect(rendered).to have_css('.sidebar-top-level-items > li.active', text: 'Registry')
end end
end end
describe 'releases entry' do
describe 'when releases feature flag is disabled' do
before do
stub_feature_flags(releases_page: false)
end
it 'does not render releases link' do
render
expect(rendered).not_to have_link('Releases', href: project_releases_path(project))
end
end
describe 'when releases feature flags is enabled' do
before do
stub_feature_flags(releases_page: true)
end
it 'renders releases link' do
render
expect(rendered).to have_link('Releases', href: project_releases_path(project))
end
end
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment