Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
1
Merge Requests
1
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
nexedi
gitlab-ce
Commits
be69dc1d
Commit
be69dc1d
authored
Mar 19, 2020
by
Douglas Barbosa Alexandre
Browse files
Options
Browse Files
Download
Plain Diff
Merge branch 'turn-off-ci-ff' into 'master'
Turn off ci_variables_ui ff See merge request gitlab-org/gitlab!27599
parents
651a786d
48382c99
Changes
5
Show whitespace changes
Inline
Side-by-side
Showing
5 changed files
with
3 additions
and
13 deletions
+3
-13
app/controllers/groups/settings/ci_cd_controller.rb
app/controllers/groups/settings/ci_cd_controller.rb
+1
-1
app/controllers/projects/settings/ci_cd_controller.rb
app/controllers/projects/settings/ci_cd_controller.rb
+1
-1
app/views/ci/variables/_index.html.haml
app/views/ci/variables/_index.html.haml
+1
-1
changelogs/unreleased/fix-ci-delete-variable-bug.yml
changelogs/unreleased/fix-ci-delete-variable-bug.yml
+0
-5
changelogs/unreleased/turn-on-new-variables-ui-ff.yml
changelogs/unreleased/turn-on-new-variables-ui-ff.yml
+0
-5
No files found.
app/controllers/groups/settings/ci_cd_controller.rb
View file @
be69dc1d
...
...
@@ -7,7 +7,7 @@ module Groups
before_action
:authorize_admin_group!
before_action
:authorize_update_max_artifacts_size!
,
only:
[
:update
]
before_action
do
push_frontend_feature_flag
(
:new_variables_ui
,
@group
,
default_enabled:
true
)
push_frontend_feature_flag
(
:new_variables_ui
,
@group
)
end
before_action
:define_variables
,
only:
[
:show
,
:create_deploy_token
]
...
...
app/controllers/projects/settings/ci_cd_controller.rb
View file @
be69dc1d
...
...
@@ -6,7 +6,7 @@ module Projects
before_action
:authorize_admin_pipeline!
before_action
:define_variables
before_action
do
push_frontend_feature_flag
(
:new_variables_ui
,
@project
,
default_enabled:
true
)
push_frontend_feature_flag
(
:new_variables_ui
,
@project
)
end
def
show
...
...
app/views/ci/variables/_index.html.haml
View file @
be69dc1d
...
...
@@ -5,7 +5,7 @@
-
link_start
=
'<a href="%{url}">'
.
html_safe
%
{
url:
help_page_path
(
'ci/variables/README'
,
anchor:
'protected-variables'
)
}
=
s_
(
'Environment variables are configured by your administrator to be %{link_start}protected%{link_end} by default'
).
html_safe
%
{
link_start:
link_start
,
link_end:
'</a>'
.
html_safe
}
-
if
Feature
.
enabled?
(
:new_variables_ui
,
@project
||
@group
,
default_enabled:
true
)
-
if
Feature
.
enabled?
(
:new_variables_ui
,
@project
||
@group
)
-
is_group
=
!
@group
.
nil?
#js-ci-project-variables
{
data:
{
endpoint:
save_endpoint
,
project_id:
@project
&
.
id
||
''
,
group:
is_group
.
to_s
,
maskable_regex:
ci_variable_maskable_regex
}
}
...
...
changelogs/unreleased/fix-ci-delete-variable-bug.yml
deleted
100644 → 0
View file @
651a786d
---
title
:
Update UI for project and group settings CI variables
merge_request
:
27411
author
:
type
:
added
changelogs/unreleased/turn-on-new-variables-ui-ff.yml
deleted
100644 → 0
View file @
651a786d
---
title
:
Update UI for project and group settings CI variables
merge_request
:
26901
author
:
type
:
added
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment