Commit 06827800 authored by Clement Ho's avatar Clement Ho

Merge branch 'tr-hide-alert-detail-dropdown' into 'master'

Remove alert detail status dropdown

See merge request gitlab-org/gitlab!31998
parents 99d21e45 b97318dd
<script> <script>
import * as Sentry from '@sentry/browser'; import * as Sentry from '@sentry/browser';
import { import { GlAlert, GlIcon, GlLoadingIcon, GlSprintf, GlTabs, GlTab, GlButton } from '@gitlab/ui';
GlAlert,
GlIcon,
GlLoadingIcon,
GlNewDropdown,
GlNewDropdownItem,
GlSprintf,
GlTabs,
GlTab,
GlButton,
} from '@gitlab/ui';
import { s__ } from '~/locale'; import { s__ } from '~/locale';
import query from '../graphql/queries/details.query.graphql'; import query from '../graphql/queries/details.query.graphql';
import { fetchPolicies } from '~/lib/graphql'; import { fetchPolicies } from '~/lib/graphql';
...@@ -38,8 +28,6 @@ export default { ...@@ -38,8 +28,6 @@ export default {
GlAlert, GlAlert,
GlIcon, GlIcon,
GlLoadingIcon, GlLoadingIcon,
GlNewDropdown,
GlNewDropdownItem,
GlSprintf, GlSprintf,
GlTab, GlTab,
GlTabs, GlTabs,
...@@ -148,15 +136,6 @@ export default { ...@@ -148,15 +136,6 @@ export default {
class="gl-display-flex gl-justify-content-space-between gl-align-items-center" class="gl-display-flex gl-justify-content-space-between gl-align-items-center"
> >
<h2 data-testid="title">{{ alert.title }}</h2> <h2 data-testid="title">{{ alert.title }}</h2>
<gl-new-dropdown right>
<gl-new-dropdown-item
v-for="(label, field) in $options.statuses"
:key="field"
data-testid="statusDropdownItem"
class="gl-vertical-align-middle"
>{{ label }}
</gl-new-dropdown-item>
</gl-new-dropdown>
</div> </div>
<gl-tabs v-if="alert" data-testid="alertDetailsTabs"> <gl-tabs v-if="alert" data-testid="alertDetailsTabs">
<gl-tab data-testid="overviewTab" :title="$options.i18n.overviewTitle"> <gl-tab data-testid="overviewTab" :title="$options.i18n.overviewTitle">
......
...@@ -110,10 +110,6 @@ describe('AlertDetails', () => { ...@@ -110,10 +110,6 @@ describe('AlertDetails', () => {
}); });
}); });
it('renders a status dropdown containing three items', () => {
expect(wrapper.findAll('[data-testid="statusDropdownItem"]').length).toBe(3);
});
describe('Create issue from alert', () => { describe('Create issue from alert', () => {
describe('createIssueFromAlertEnabled feature flag enabled', () => { describe('createIssueFromAlertEnabled feature flag enabled', () => {
it('should display a button that links to new issue page', () => { it('should display a button that links to new issue page', () => {
......
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