Commit a52cd764 authored by Phil Hughes's avatar Phil Hughes

Merge branch '3441-group-board-labels' into 'master'

show project labels on group board cards

Closes #3441

See merge request gitlab-org/gitlab-ee!2957
parents c653ae39 f24d6aee
...@@ -111,7 +111,6 @@ gl.issueBoards.IssueCardInner = Vue.extend({ ...@@ -111,7 +111,6 @@ gl.issueBoards.IssueCardInner = Vue.extend({
}, },
showLabel(label) { showLabel(label) {
if (!label.id) return false; if (!label.id) return false;
if (this.groupId && label.type === 'ProjectLabel') return false;
return true; return true;
}, },
filterByLabel(label, e) { filterByLabel(label, e) {
......
...@@ -322,7 +322,7 @@ describe('Issue card component', () => { ...@@ -322,7 +322,7 @@ describe('Issue card component', () => {
.catch(done.fail); .catch(done.fail);
}); });
it('does not show project labels on group boards', (done) => { it('shows project labels on group boards', (done) => {
component.issue.addLabel(new ListLabel({ component.issue.addLabel(new ListLabel({
id: 123, id: 123,
title: 'Project label', title: 'Project label',
...@@ -334,10 +334,10 @@ describe('Issue card component', () => { ...@@ -334,10 +334,10 @@ describe('Issue card component', () => {
.then(() => { .then(() => {
expect( expect(
component.$el.querySelectorAll('.label').length, component.$el.querySelectorAll('.label').length,
).toBe(2); ).toBe(3);
expect( expect(
component.$el.textContent, component.$el.textContent,
).not.toContain('Project label'); ).toContain('Project label');
done(); done();
}) })
......
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