Skip to content

Commit f34b26b

Browse files
author
GitLab Bot
committed
Add latest changes from gitlab-org/gitlab@master
1 parent 04dabf4 commit f34b26b

File tree

46 files changed

+1123
-68
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

46 files changed

+1123
-68
lines changed

Gemfile

+1-1
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,7 @@ gem 'prometheus-client-mmap', '~> 0.17', require: 'prometheus/client'
373373
gem 'warning', '~> 1.3.0'
374374

375375
group :development do
376-
gem 'lefthook', '~> 1.2.9', require: false
376+
gem 'lefthook', '~> 1.3.0', require: false
377377
gem 'rubocop'
378378
gem 'solargraph', '~> 0.47.2', require: false
379379

Gemfile.checksum

+1-1
Original file line numberDiff line numberDiff line change
@@ -314,7 +314,7 @@
314314
{"name":"kramdown","version":"2.3.2","platform":"ruby","checksum":"cb4530c2e9d16481591df2c9336723683c354e5416a5dd3e447fa48215a6a71c"},
315315
{"name":"kramdown-parser-gfm","version":"1.1.0","platform":"ruby","checksum":"fb39745516427d2988543bf01fc4cf0ab1149476382393e0e9c48592f6581729"},
316316
{"name":"launchy","version":"2.5.0","platform":"ruby","checksum":"954243c4255920982ce682f89a42e76372dba94770bf09c23a523e204bdebef5"},
317-
{"name":"lefthook","version":"1.2.9","platform":"ruby","checksum":"1fd4a768e08fc624e756597fc628b3c7991267325974a7a5cc169595b425701d"},
317+
{"name":"lefthook","version":"1.3.0","platform":"ruby","checksum":"46460ceb0084d1a60c7aa2872c90fd9a97d92c32063b41ac88303e1d1a382b43"},
318318
{"name":"letter_opener","version":"1.7.0","platform":"ruby","checksum":"095bc0d58e006e5b43ea7d219e64ecf2de8d1f7d9dafc432040a845cf59b4725"},
319319
{"name":"letter_opener_web","version":"2.0.0","platform":"ruby","checksum":"33860ad41e1785d75456500e8ca8bba8ed71ee6eaf08a98d06bbab67c5577b6f"},
320320
{"name":"libyajl2","version":"1.2.0","platform":"ruby","checksum":"1117cd1e48db013b626e36269bbf1cef210538ca6d2e62d3fa3db9ded005b258"},

Gemfile.lock

+3-3
Original file line numberDiff line numberDiff line change
@@ -845,7 +845,7 @@ GEM
845845
kramdown (~> 2.0)
846846
launchy (2.5.0)
847847
addressable (~> 2.7)
848-
lefthook (1.2.9)
848+
lefthook (1.3.0)
849849
letter_opener (1.7.0)
850850
launchy (~> 2.2)
851851
letter_opener_web (2.0.0)
@@ -1738,7 +1738,7 @@ DEPENDENCIES
17381738
knapsack (~> 1.21.1)
17391739
kramdown (~> 2.3.1)
17401740
kubeclient (~> 4.9.3)!
1741-
lefthook (~> 1.2.9)
1741+
lefthook (~> 1.3.0)
17421742
letter_opener_web (~> 2.0.0)
17431743
license_finder (~> 7.0)
17441744
licensee (~> 9.15)
@@ -1893,4 +1893,4 @@ DEPENDENCIES
18931893
yajl-ruby (~> 1.4.3)
18941894

18951895
BUNDLED WITH
1896-
2.4.6
1896+
2.4.7

app/assets/javascripts/packages_and_registries/package_registry/components/details/package_versions_list.vue

+40
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,22 @@ import VersionRow from '~/packages_and_registries/package_registry/components/de
44
import PackagesListLoader from '~/packages_and_registries/shared/components/packages_list_loader.vue';
55
import RegistryList from '~/packages_and_registries/shared/components/registry_list.vue';
66
import DeleteModal from '~/packages_and_registries/package_registry/components/delete_modal.vue';
7+
import DeletePackageModal from '~/packages_and_registries/shared/components/delete_package_modal.vue';
78
import {
9+
CANCEL_DELETE_PACKAGE_VERSION_TRACKING_ACTION,
810
CANCEL_DELETE_PACKAGE_VERSIONS_TRACKING_ACTION,
11+
DELETE_PACKAGE_VERSION_TRACKING_ACTION,
912
DELETE_PACKAGE_VERSIONS_TRACKING_ACTION,
13+
REQUEST_DELETE_PACKAGE_VERSION_TRACKING_ACTION,
1014
REQUEST_DELETE_PACKAGE_VERSIONS_TRACKING_ACTION,
1115
} from '~/packages_and_registries/package_registry/constants';
1216
import Tracking from '~/tracking';
17+
import { packageTypeToTrackCategory } from '~/packages_and_registries/package_registry/utils';
1318
1419
export default {
1520
components: {
1621
DeleteModal,
22+
DeletePackageModal,
1723
VersionRow,
1824
PackagesListLoader,
1925
RegistryList,
@@ -42,6 +48,7 @@ export default {
4248
},
4349
data() {
4450
return {
51+
itemToBeDeleted: null,
4552
itemsToBeDeleted: [],
4653
};
4754
},
@@ -52,8 +59,25 @@ export default {
5259
isListEmpty() {
5360
return this.versions.length === 0;
5461
},
62+
tracking() {
63+
const category = this.itemToBeDeleted
64+
? packageTypeToTrackCategory(this.itemToBeDeleted.packageType)
65+
: undefined;
66+
return {
67+
category,
68+
};
69+
},
5570
},
5671
methods: {
72+
deleteItemConfirmation() {
73+
this.$emit('delete', [this.itemToBeDeleted]);
74+
this.track(DELETE_PACKAGE_VERSION_TRACKING_ACTION);
75+
this.itemToBeDeleted = null;
76+
},
77+
deleteItemCanceled() {
78+
this.track(CANCEL_DELETE_PACKAGE_VERSION_TRACKING_ACTION);
79+
this.itemToBeDeleted = null;
80+
},
5781
deleteItemsCanceled() {
5882
this.track(CANCEL_DELETE_PACKAGE_VERSIONS_TRACKING_ACTION);
5983
this.itemsToBeDeleted = [];
@@ -63,7 +87,16 @@ export default {
6387
this.track(DELETE_PACKAGE_VERSIONS_TRACKING_ACTION);
6488
this.itemsToBeDeleted = [];
6589
},
90+
setItemToBeDeleted(item) {
91+
this.itemToBeDeleted = { ...item };
92+
this.track(REQUEST_DELETE_PACKAGE_VERSION_TRACKING_ACTION);
93+
},
6694
setItemsToBeDeleted(items) {
95+
if (items.length === 1) {
96+
const [item] = items;
97+
this.setItemToBeDeleted(item);
98+
return;
99+
}
67100
this.itemsToBeDeleted = items;
68101
this.track(REQUEST_DELETE_PACKAGE_VERSIONS_TRACKING_ACTION);
69102
this.$refs.deletePackagesModal.show();
@@ -96,11 +129,18 @@ export default {
96129
:first="canDestroy && first"
97130
:package-entity="item"
98131
:selected="isSelected(item)"
132+
@delete="setItemToBeDeleted(item)"
99133
@select="selectItem(item)"
100134
/>
101135
</template>
102136
</registry-list>
103137

138+
<delete-package-modal
139+
:item-to-be-deleted="itemToBeDeleted"
140+
@ok="deleteItemConfirmation"
141+
@cancel="deleteItemCanceled"
142+
/>
143+
104144
<delete-modal
105145
ref="deletePackagesModal"
106146
:items-to-be-deleted="itemsToBeDeleted"

app/assets/javascripts/packages_and_registries/package_registry/components/details/version_row.vue

+20
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
<script>
22
import {
3+
GlDropdown,
4+
GlDropdownItem,
35
GlFormCheckbox,
46
GlIcon,
57
GlLink,
@@ -13,6 +15,7 @@ import PublishMethod from '~/packages_and_registries/shared/components/publish_m
1315
import ListItem from '~/vue_shared/components/registry/list_item.vue';
1416
import TimeAgoTooltip from '~/vue_shared/components/time_ago_tooltip.vue';
1517
import {
18+
DELETE_PACKAGE_TEXT,
1619
ERRORED_PACKAGE_TEXT,
1720
ERROR_PUBLISHING,
1821
PACKAGE_ERROR_STATUS,
@@ -22,6 +25,8 @@ import {
2225
export default {
2326
name: 'PackageVersionRow',
2427
components: {
28+
GlDropdown,
29+
GlDropdownItem,
2530
GlFormCheckbox,
2631
GlIcon,
2732
GlLink,
@@ -58,6 +63,7 @@ export default {
5863
},
5964
},
6065
i18n: {
66+
deletePackage: DELETE_PACKAGE_TEXT,
6167
erroredPackageText: ERRORED_PACKAGE_TEXT,
6268
errorPublishing: ERROR_PUBLISHING,
6369
warningText: WARNING_TEXT,
@@ -121,5 +127,19 @@ export default {
121127
</gl-sprintf>
122128
</span>
123129
</template>
130+
131+
<template v-if="packageEntity.canDestroy" #right-action>
132+
<gl-dropdown
133+
icon="ellipsis_v"
134+
:text="$options.i18n.moreActions"
135+
:text-sr-only="true"
136+
category="tertiary"
137+
no-caret
138+
>
139+
<gl-dropdown-item variant="danger" @click="$emit('delete')">{{
140+
$options.i18n.deletePackage
141+
}}</gl-dropdown-item>
142+
</gl-dropdown>
143+
</template>
124144
</list-item>
125145
</template>

app/assets/javascripts/packages_and_registries/package_registry/components/list/package_list_row.vue

+3-2
Original file line numberDiff line numberDiff line change
@@ -8,9 +8,10 @@ import {
88
GlTooltipDirective,
99
GlTruncate,
1010
} from '@gitlab/ui';
11-
import { s__, __ } from '~/locale';
11+
import { __ } from '~/locale';
1212
import ListItem from '~/vue_shared/components/registry/list_item.vue';
1313
import {
14+
DELETE_PACKAGE_TEXT,
1415
ERRORED_PACKAGE_TEXT,
1516
ERROR_PUBLISHING,
1617
PACKAGE_ERROR_STATUS,
@@ -91,7 +92,7 @@ export default {
9192
i18n: {
9293
erroredPackageText: ERRORED_PACKAGE_TEXT,
9394
createdAt: __('Created %{timestamp}'),
94-
deletePackage: s__('PackageRegistry|Delete package'),
95+
deletePackage: DELETE_PACKAGE_TEXT,
9596
errorPublishing: ERROR_PUBLISHING,
9697
warning: WARNING_TEXT,
9798
moreActions: __('More actions'),

app/assets/javascripts/packages_and_registries/package_registry/constants.js

+5
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,10 @@ export const DELETE_PACKAGE_VERSIONS_TRACKING_ACTION = 'delete_package_versions'
119119
export const REQUEST_DELETE_PACKAGE_VERSIONS_TRACKING_ACTION = 'request_delete_package_versions';
120120
export const CANCEL_DELETE_PACKAGE_VERSIONS_TRACKING_ACTION = 'cancel_delete_package_versions';
121121

122+
export const DELETE_PACKAGE_VERSION_TRACKING_ACTION = 'delete_package_version';
123+
export const REQUEST_DELETE_PACKAGE_VERSION_TRACKING_ACTION = 'request_delete_package_version';
124+
export const CANCEL_DELETE_PACKAGE_VERSION_TRACKING_ACTION = 'cancel_delete_package_version';
125+
122126
export const DELETE_PACKAGES_ERROR_MESSAGE = s__(
123127
'PackageRegistry|Something went wrong while deleting packages.',
124128
);
@@ -127,6 +131,7 @@ export const DELETE_PACKAGES_SUCCESS_MESSAGE = s__('PackageRegistry|Packages del
127131
export const DELETE_PACKAGES_MODAL_TITLE = s__('PackageRegistry|Delete packages');
128132
export const DELETE_PACKAGE_MODAL_PRIMARY_ACTION = s__('PackageRegistry|Permanently delete');
129133

134+
export const DELETE_PACKAGE_TEXT = s__('PackageRegistry|Delete package');
130135
export const DELETE_PACKAGE_SUCCESS_MESSAGE = s__('PackageRegistry|Package deleted successfully');
131136
export const DELETE_PACKAGE_ERROR_MESSAGE = s__(
132137
'PackageRegistry|Something went wrong while deleting the package.',

0 commit comments

Comments
 (0)