Skip to content

Commit d62742b

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent e18006f commit d62742b

File tree

38 files changed

+553
-115
lines changed

38 files changed

+553
-115
lines changed

.gitignore

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,3 +117,6 @@ jest-snapshot-test-report.json
117117
# https://vitejs.dev/guide/env-and-mode.html#env-files
118118
*.local
119119
/config/vite.gdk.json
120+
121+
# CSS compilation for cssbundling
122+
app/assets/builds/

.gitlab/ci/rules.gitlab-ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1152,15 +1152,15 @@
11521152
# If any of the files that are DIRECTLY related to generating or managing the GLFM specification change,
11531153
# run `glfm-verify` to get quick feedback on any needed updates, even if the MR is not yet approved
11541154
- <<: *if-default-refs
1155-
- changes: *glfm-patterns
1155+
changes: *glfm-patterns
11561156
# Otherwise do not run `glfm-verify` if the MR is not approved
11571157
- <<: *if-merge-request-not-approved
11581158
when: never
11591159
# If we passed all the previous rules, run `glfm-verify` if there are any changes that could impact `glfm-verify`.
11601160
# This could potentially be a wide range of files, so we reuse `setup-test-env-patterns`, which includes
11611161
# almost all app files except docs files.
11621162
- <<: *if-default-refs
1163-
- changes: *setup-test-env-patterns
1163+
changes: *setup-test-env-patterns
11641164
# If we are forcing all rspec to run, run this job too.
11651165
- <<: *if-merge-request-labels-run-all-rspec
11661166

GITALY_SERVER_VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
a22e2401ee6aa35ae70ca67013264b72fc29b6f5
1+
7eb79ebcb084d4e881777f44ca5055cce6e60ccf

app/assets/javascripts/editor/schema/ci.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1750,6 +1750,9 @@
17501750
"project",
17511751
"ref"
17521752
]
1753+
},
1754+
{
1755+
"$ref": "#/definitions/!reference"
17531756
}
17541757
]
17551758
}
Lines changed: 55 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,55 @@
1+
<script>
2+
import UsageBanner from '~/vue_shared/components/usage_quotas/usage_banner.vue';
3+
import { s__ } from '~/locale';
4+
import NumberToHumanSize from '~/vue_shared/components/number_to_human_size/number_to_human_size.vue';
5+
import HelpPageLink from '~/vue_shared/components/help_page_link/help_page_link.vue';
6+
7+
export default {
8+
name: 'DependencyProxyUsage',
9+
components: {
10+
NumberToHumanSize,
11+
HelpPageLink,
12+
UsageBanner,
13+
},
14+
props: {
15+
dependencyProxyTotalSize: {
16+
type: Number,
17+
required: false,
18+
default: 0,
19+
},
20+
loading: {
21+
type: Boolean,
22+
required: false,
23+
default: false,
24+
},
25+
},
26+
i18n: {
27+
dependencyProxy: s__('UsageQuota|Dependency proxy'),
28+
storageUsed: s__('UsageQuota|Storage used'),
29+
dependencyProxyMessage: s__(
30+
'UsageQuota|Local proxy used for frequently-accessed upstream Docker images.',
31+
),
32+
},
33+
};
34+
</script>
35+
<template>
36+
<usage-banner :loading="loading">
37+
<template #left-primary-text>
38+
{{ $options.i18n.dependencyProxy }}
39+
</template>
40+
<template #left-secondary-text>
41+
<div data-testid="dependency-proxy-description">
42+
{{ $options.i18n.dependencyProxyMessage }}
43+
<help-page-link href="user/packages/dependency_proxy/index">
44+
{{ __('More information') }}
45+
</help-page-link>
46+
</div>
47+
</template>
48+
<template #right-primary-text>
49+
{{ $options.i18n.storageUsed }}
50+
</template>
51+
<template #right-secondary-text>
52+
<number-to-human-size :value="dependencyProxyTotalSize" data-testid="dependency-proxy-size" />
53+
</template>
54+
</usage-banner>
55+
</template>

app/assets/javascripts/usage_quotas/storage/components/namespace_storage_app.vue

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,16 @@
11
<script>
22
import { GlAlert } from '@gitlab/ui';
33
import StorageUsageStatistics from 'ee_else_ce/usage_quotas/storage/components/storage_usage_statistics.vue';
4+
import DependencyProxyUsage from '~/usage_quotas/storage/components/dependency_proxy_usage.vue';
45
56
export default {
67
name: 'NamespaceStorageApp',
78
components: {
89
GlAlert,
910
StorageUsageStatistics,
11+
DependencyProxyUsage,
1012
},
13+
inject: ['userNamespace'],
1114
props: {
1215
namespaceLoadingError: {
1316
type: Boolean,
@@ -39,6 +42,9 @@ export default {
3942
this.namespace.rootStorageStatistics?.storageSize
4043
);
4144
},
45+
dependencyProxyTotalSize() {
46+
return this.namespace.rootStorageStatistics?.dependencyProxySize ?? 0;
47+
},
4248
},
4349
};
4450
</script>
@@ -61,6 +67,14 @@ export default {
6167
:used-storage="usedStorage"
6268
:loading="isNamespaceStorageStatisticsLoading"
6369
/>
70+
<h3 data-testid="breakdown-subtitle">
71+
{{ s__('UsageQuota|Storage usage breakdown') }}
72+
</h3>
73+
<dependency-proxy-usage
74+
v-if="!userNamespace"
75+
:dependency-proxy-total-size="dependencyProxyTotalSize"
76+
:loading="isNamespaceStorageStatisticsLoading"
77+
/>
6478
6579
<slot name="ee-storage-app"></slot>
6680
</div>

app/assets/javascripts/work_items/components/work_item_links/index.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
import Vue from 'vue';
22
import { GlToast } from '@gitlab/ui';
3+
import { parseBoolean } from '~/lib/utils/common_utils';
34
import { apolloProvider } from '~/graphql_shared/issuable_client';
45
import WorkItemLinks from './work_item_links.vue';
56

@@ -20,6 +21,7 @@ export default function initWorkItemLinks() {
2021
registerPath,
2122
signInPath,
2223
wiReportAbusePath,
24+
isGroup,
2325
} = workItemLinksRoot.dataset;
2426

2527
return new Vue({
@@ -34,6 +36,7 @@ export default function initWorkItemLinks() {
3436
registerPath,
3537
signInPath,
3638
reportAbusePath: wiReportAbusePath,
39+
isGroup: parseBoolean(isGroup),
3740
},
3841
render: (createElement) =>
3942
createElement(WorkItemLinks, {

app/assets/stylesheets/page_bundles/issuable_list.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@
9090
.issuable-list li,
9191
.issuable-info-container .controls {
9292
.avatar-counter {
93-
@include gl-pl-1
93+
@include gl-pl-1;
9494
@include gl-pr-2;
9595
@include gl-h-5;
9696
@include gl-min-w-5;

app/controllers/admin/users_controller.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ class Admin::UsersController < Admin::ApplicationController
1010
before_action :set_shared_view_parameters, only: [:show, :projects, :keys]
1111

1212
before_action only: [:index] do
13-
push_frontend_feature_flag(:simplified_badges)
13+
push_frontend_feature_flag(:simplified_badges, current_user)
1414
end
1515

1616
feature_category :user_management

app/views/admin/spam_logs/_spam_log.html.haml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,8 @@
2929
variant: :danger,
3030
method: :delete,
3131
href: admin_spam_log_path(spam_log, remove_user: true),
32-
button_options: { data: { confirm: _("User %{user_name} will be removed! Are you sure?") % { user_name: user.name }, confirm_btn_variant: 'danger' }, aria: { label: _('Remove user') } }) do
32+
button_options: { class: ' gl-mb-3', data: { confirm: _("User %{user_name} will be removed! Are you sure?") % { user_name: user.name }, confirm_btn_variant: 'danger' } }) do
3333
= _('Remove user')
34-
%td
3534
-# TODO: Remove conditonal once spamcheck supports this https://gitlab.com/gitlab-com/gl-security/engineering-and-research/automation-team/spam/spamcheck/-/issues/190
3635
- if akismet_enabled?
3736
- if spam_log.submitted_as_ham?

0 commit comments

Comments
 (0)