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
3e053379
Commit
3e053379
authored
Jul 23, 2020
by
Olena Horal-Koretska
Committed by
Phil Hughes
Jul 23, 2020
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Display assignees in Incident List
parent
63b2e6c4
Changes
6
Hide whitespace changes
Inline
Side-by-side
Showing
6 changed files
with
89 additions
and
9 deletions
+89
-9
app/assets/javascripts/incidents/components/incidents_list.vue
...ssets/javascripts/incidents/components/incidents_list.vue
+43
-7
app/assets/javascripts/incidents/constants.js
app/assets/javascripts/incidents/constants.js
+1
-0
app/assets/javascripts/incidents/graphql/queries/get_incidents.query.graphql
...pts/incidents/graphql/queries/get_incidents.query.graphql
+3
-0
changelogs/unreleased/229530-assignee-in-incidents-list.yml
changelogs/unreleased/229530-assignee-in-incidents-list.yml
+5
-0
spec/frontend/incidents/components/incidents_list_spec.js
spec/frontend/incidents/components/incidents_list_spec.js
+27
-1
spec/frontend/incidents/mocks/incidents.json
spec/frontend/incidents/mocks/incidents.json
+10
-1
No files found.
app/assets/javascripts/incidents/components/incidents_list.vue
View file @
3e053379
<
script
>
import
{
GlLoadingIcon
,
GlTable
,
GlAlert
}
from
'
@gitlab/ui
'
;
import
{
GlLoadingIcon
,
GlTable
,
GlAlert
,
GlAvatarsInline
,
GlAvatarLink
,
GlAvatar
,
GlTooltipDirective
,
}
from
'
@gitlab/ui
'
;
import
TimeAgoTooltip
from
'
~/vue_shared/components/time_ago_tooltip.vue
'
;
import
{
s__
}
from
'
~/locale
'
;
import
getIncidents
from
'
../graphql/queries/get_incidents.query.graphql
'
;
...
...
@@ -37,8 +45,14 @@ export default {
GlLoadingIcon
,
GlTable
,
GlAlert
,
GlAvatarsInline
,
GlAvatarLink
,
GlAvatar
,
TimeAgoTooltip
,
},
directives
:
{
GlTooltip
:
GlTooltipDirective
,
},
inject
:
[
'
projectPath
'
],
apollo
:
{
incidents
:
{
...
...
@@ -78,10 +92,8 @@ export default {
},
},
methods
:
{
getAssignees
(
assignees
)
{
return
assignees
.
nodes
?.
length
>
0
?
assignees
.
nodes
[
0
]?.
username
:
s__
(
'
IncidentManagement|Unassigned
'
);
hasAssignees
(
assignees
)
{
return
Boolean
(
assignees
.
nodes
?.
length
);
},
},
};
...
...
@@ -114,8 +126,32 @@ export default {
</
template
>
<
template
#cell(assignees)=
"{ item }"
>
<div
class=
"gl-max-w-full text-truncate"
data-testid=
"assigneesField"
>
{{
getAssignees
(
item
.
assignees
)
}}
<div
data-testid=
"incident-assignees"
>
<template
v-if=
"hasAssignees(item.assignees)"
>
<gl-avatars-inline
:avatars=
"item.assignees.nodes"
:collapsed=
"true"
:max-visible=
"4"
:avatar-size=
"24"
badge-tooltip-prop=
"name"
:badge-tooltip-max-chars=
"100"
>
<template
#avatar
="
{ avatar }">
<gl-avatar-link
:key=
"avatar.username"
v-gl-tooltip
target=
"_blank"
:href=
"avatar.webUrl"
:title=
"avatar.name"
>
<gl-avatar
:src=
"avatar.avatarUrl"
:label=
"avatar.name"
:size=
"24"
/>
</gl-avatar-link>
</
template
>
</gl-avatars-inline>
</template>
<
template
v-else
>
{{
$options
.
i18n
.
unassigned
}}
</
template
>
</div>
</template>
...
...
app/assets/javascripts/incidents/constants.js
View file @
3e053379
...
...
@@ -4,4 +4,5 @@ import { s__ } from '~/locale';
export
const
I18N
=
{
errorMsg
:
s__
(
'
IncidentManagement|There was an error displaying the incidents.
'
),
noIncidents
:
s__
(
'
IncidentManagement|No incidents to display.
'
),
unassigned
:
s__
(
'
IncidentManagement|Unassigned
'
),
};
app/assets/javascripts/incidents/graphql/queries/get_incidents.query.graphql
View file @
3e053379
...
...
@@ -13,7 +13,10 @@ query getIncidents($projectPath: ID!, $labelNames: [String], $state: IssuableSta
}
assignees
{
nodes
{
name
username
avatarUrl
webUrl
}
}
}
...
...
changelogs/unreleased/229530-assignee-in-incidents-list.yml
0 → 100644
View file @
3e053379
---
title
:
Display assignees in Incident List
merge_request
:
37608
author
:
type
:
added
spec/frontend/incidents/components/incidents_list_spec.js
View file @
3e053379
import
{
mount
}
from
'
@vue/test-utils
'
;
import
{
GlAlert
,
GlLoadingIcon
,
GlTable
}
from
'
@gitlab/ui
'
;
import
{
GlAlert
,
GlLoadingIcon
,
GlTable
,
GlAvatar
}
from
'
@gitlab/ui
'
;
import
IncidentsList
from
'
~/incidents/components/incidents_list.vue
'
;
import
TimeAgoTooltip
from
'
~/vue_shared/components/time_ago_tooltip.vue
'
;
import
{
I18N
}
from
'
~/incidents/constants
'
;
...
...
@@ -13,6 +13,7 @@ describe('Incidents List', () => {
const
findAlert
=
()
=>
wrapper
.
find
(
GlAlert
);
const
findLoader
=
()
=>
wrapper
.
find
(
GlLoadingIcon
);
const
findTimeAgo
=
()
=>
wrapper
.
findAll
(
TimeAgoTooltip
);
const
findAssingees
=
()
=>
wrapper
.
findAll
(
'
[data-testid="incident-assignees"]
'
);
function
mountComponent
({
data
=
{
incidents
:
[]
},
loading
=
false
})
{
wrapper
=
mount
(
IncidentsList
,
{
...
...
@@ -31,6 +32,9 @@ describe('Incidents List', () => {
provide
:
{
projectPath
:
'
/project/path
'
,
},
stubs
:
{
GlAvatar
:
true
,
},
});
}
...
...
@@ -80,5 +84,27 @@ describe('Incidents List', () => {
it
(
'
renders a createdAt with timeAgo component per row
'
,
()
=>
{
expect
(
findTimeAgo
().
length
).
toBe
(
mockIncidents
.
length
);
});
describe
(
'
Assignees
'
,
()
=>
{
it
(
'
shows Unassigned when there are no assignees
'
,
()
=>
{
expect
(
findAssingees
()
.
at
(
0
)
.
text
(),
).
toBe
(
I18N
.
unassigned
);
});
it
(
'
renders an avatar component when there is an assignee
'
,
()
=>
{
const
avatar
=
findAssingees
()
.
at
(
1
)
.
find
(
GlAvatar
);
const
{
src
,
label
}
=
avatar
.
attributes
();
const
{
name
,
avatarUrl
}
=
mockIncidents
[
1
].
assignees
.
nodes
[
0
];
expect
(
avatar
.
exists
()).
toBe
(
true
);
expect
(
label
).
toBe
(
name
);
expect
(
src
).
toBe
(
avatarUrl
);
});
});
});
});
spec/frontend/incidents/mocks/incidents.json
View file @
3e053379
...
...
@@ -9,7 +9,16 @@
"iid"
:
"14"
,
"title"
:
"Create issue4"
,
"createdAt"
:
"2020-05-19T09:26:07Z"
,
"assignees"
:
{}
"assignees"
:
{
"nodes"
:
[
{
"name"
:
"Benjamin Braun"
,
"username"
:
"kami.hegmann"
,
"avatarUrl"
:
"https://invalid'"
,
"webUrl"
:
"https://invalid"
}
]
}
},
{
"iid"
:
"13"
,
...
...
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