Commit 83e98e05 authored by Justin Ho's avatar Justin Ho

Remove frontend for editing of Jira labels

Feature is will no longer be implemented

Changelog: removed
EE: true
parent 829d5921
......@@ -35,7 +35,6 @@ export default {
return {
isLoading: true,
isLoadingStatus: false,
isUpdatingLabels: false,
isUpdatingStatus: false,
errorMessage: null,
issue: {},
......@@ -79,23 +78,6 @@ export default {
return `jira_note_${id}`;
},
onIssueLabelsUpdated(labels) {
this.isUpdatingLabels = true;
updateIssue(this.issue, { labels })
.then((response) => {
this.issue.labels = response.labels;
})
.catch(() => {
createFlash({
message: s__(
'JiraService|Failed to update Jira issue labels. View the issue in Jira, or reload the page.',
),
});
})
.finally(() => {
this.isUpdatingLabels = false;
});
},
onIssueStatusFetch() {
this.isLoadingStatus = true;
fetchIssueStatuses()
......@@ -156,10 +138,8 @@ export default {
:sidebar-expanded="sidebarExpanded"
:issue="issue"
:is-loading-status="isLoadingStatus"
:is-updating-labels="isUpdatingLabels"
:is-updating-status="isUpdatingStatus"
:statuses="statuses"
@issue-labels-updated="onIssueLabelsUpdated"
@issue-status-fetch="onIssueStatusFetch"
@issue-status-updated="onIssueStatusUpdated"
@sidebar-toggle="toggleSidebar"
......
......@@ -20,9 +20,6 @@ export default {
},
mixins: [glFeatureFlagsMixin()],
inject: {
issueLabelsPath: {
default: null,
},
issuesListPath: {
default: null,
},
......@@ -41,11 +38,6 @@ export default {
required: false,
default: false,
},
isUpdatingLabels: {
type: Boolean,
required: false,
default: false,
},
isUpdatingStatus: {
type: Boolean,
required: false,
......@@ -57,11 +49,6 @@ export default {
default: () => [],
},
},
data() {
return {
isEditingLabels: false,
};
},
computed: {
assignee() {
// Jira issues have at most 1 assignee
......@@ -70,9 +57,6 @@ export default {
reference() {
return this.issue.references?.relative;
},
canUpdateLabels() {
return this.glFeatures.jiraIssueDetailsEditLabels;
},
canUpdateStatus() {
return this.glFeatures.jiraIssueDetailsEditStatus;
},
......@@ -108,18 +92,6 @@ export default {
});
}
},
onIssueLabelsClose() {
this.isEditingLabels = false;
},
onIssueLabelsToggle() {
this.expandSidebarAndOpenDropdown();
this.afterSidebarTransitioned(() => {
this.isEditingLabels = true;
});
},
onIssueLabelsUpdated(labels) {
this.$emit('issue-labels-updated', labels);
},
onIssueStatusFetch() {
this.$emit('issue-status-fetch');
},
......@@ -149,20 +121,13 @@ export default {
@issue-field-updated="onIssueStatusUpdated"
/>
<labels-select
:allow-label-edit="canUpdateLabels"
:allow-multiselect="true"
:allow-scoped-labels="true"
:selected-labels="issue.labels"
:labels-fetch-path="issueLabelsPath"
:labels-filter-base-path="issuesListPath"
:labels-filter-param="$options.labelsFilterParam"
:labels-select-in-progress="isUpdatingLabels"
:is-editing="isEditingLabels"
variant="sidebar"
class="block labels js-labels-block"
@onDropdownClose="onIssueLabelsClose"
@toggleCollapse="onIssueLabelsToggle"
@updateSelectedLabels="onIssueLabelsUpdated"
@toggleCollapse="expandSidebarAndOpenDropdown"
>
{{ __('None') }}
</labels-select>
......
......@@ -9,12 +9,11 @@ export default function initJiraIssueShow({ mountPointSelector }) {
return null;
}
const { issueLabelsPath, issuesShowPath, issuesListPath } = mountPointEl.dataset;
const { issuesShowPath, issuesListPath } = mountPointEl.dataset;
return new Vue({
el: mountPointEl,
provide: {
issueLabelsPath,
issuesShowPath,
issuesListPath,
isClassicSidebar: true,
......
......@@ -120,22 +120,6 @@ describe('JiraIssuesShow', () => {
await waitForPromises();
});
it('updates issue labels on issue-labels-updated', async () => {
const updateIssueSpy = jest.spyOn(JiraIssuesShowApi, 'updateIssue').mockResolvedValue();
const labels = [{ id: 'ecosystem' }];
findJiraIssueSidebar().vm.$emit('issue-labels-updated', labels);
await wrapper.vm.$nextTick();
expect(updateIssueSpy).toHaveBeenCalledWith(expect.any(Object), { labels });
expect(findJiraIssueSidebar().props('isUpdatingLabels')).toBe(true);
await waitForPromises();
expect(findJiraIssueSidebar().props('isUpdatingLabels')).toBe(false);
});
it('fetches issue statuses on issue-status-fetch', async () => {
const fetchIssueStatusesSpy = jest
.spyOn(JiraIssuesShowApi, 'fetchIssueStatuses')
......
......@@ -19607,9 +19607,6 @@ msgstr ""
msgid "JiraService|Failed to load Jira issue. View the issue in Jira, or reload the page."
msgstr ""
msgid "JiraService|Failed to update Jira issue labels. View the issue in Jira, or reload the page."
msgstr ""
msgid "JiraService|Failed to update Jira issue status. View the issue in Jira, or reload the page."
msgstr ""
......
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