summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPhil Hughes <me@iamphill.com>2018-12-21 11:53:03 +0000
committerPhil Hughes <me@iamphill.com>2018-12-21 11:53:03 +0000
commit7f334fdfafd5ea8a1c4277dcf4b1cb56e79c529b (patch)
tree99af3de1b52bce650efe65d22413f43faa17aa54
parent84655c3fd9f81b7c53b541ff920d385712125016 (diff)
parent61dd59464fca96092c8bb1b7dd9878e3c6b22a1b (diff)
downloadgitlab-ce-7f334fdfafd5ea8a1c4277dcf4b1cb56e79c529b.tar.gz
Merge branch '55639-releases' into 'master'
Resolve "Wrong API call on releases page" Closes #55639 See merge request gitlab-org/gitlab-ce!23978
-rw-r--r--app/assets/javascripts/api.js2
-rw-r--r--app/assets/javascripts/releases/components/release_block.vue19
-rw-r--r--app/assets/javascripts/releases/index.js2
-rw-r--r--spec/javascripts/releases/components/release_block_spec.js20
4 files changed, 25 insertions, 18 deletions
diff --git a/app/assets/javascripts/api.js b/app/assets/javascripts/api.js
index a1310d18c26..d1396b6c4bc 100644
--- a/app/assets/javascripts/api.js
+++ b/app/assets/javascripts/api.js
@@ -29,7 +29,7 @@ const Api = {
commitPipelinesPath: '/:project_id/commit/:sha/pipelines',
branchSinglePath: '/api/:version/projects/:id/repository/branches/:branch',
createBranchPath: '/api/:version/projects/:id/repository/branches',
- releasesPath: '/api/:version/project/:id/releases',
+ releasesPath: '/api/:version/projects/:id/releases',
group(groupId, callback) {
const url = Api.buildUrl(Api.groupPath).replace(':id', groupId);
diff --git a/app/assets/javascripts/releases/components/release_block.vue b/app/assets/javascripts/releases/components/release_block.vue
index 9c2aade51fc..34b97826cdb 100644
--- a/app/assets/javascripts/releases/components/release_block.vue
+++ b/app/assets/javascripts/releases/components/release_block.vue
@@ -1,4 +1,5 @@
<script>
+import _ from 'underscore';
import { GlTooltipDirective, GlLink } from '@gitlab/ui';
import Icon from '~/vue_shared/components/icon.vue';
import UserAvatarLink from '~/vue_shared/components/user_avatar/user_avatar_link.vue';
@@ -30,8 +31,8 @@ export default {
});
},
userImageAltDescription() {
- return this.commit.author && this.commit.author.username
- ? sprintf("%{username}'s avatar", { username: this.commit.author.username })
+ return this.author && this.author.username
+ ? sprintf("%{username}'s avatar", { username: this.author.username })
: null;
},
commit() {
@@ -40,6 +41,12 @@ export default {
assets() {
return this.release.assets || {};
},
+ author() {
+ return this.release.author || {};
+ },
+ hasAuthor() {
+ return _.isEmpty(this.author);
+ },
},
};
</script>
@@ -66,14 +73,14 @@ export default {
}}</span>
</div>
- <div v-if="commit.author" class="d-flex">
+ <div v-if="hasAuthor" class="d-flex">
by
<user-avatar-link
class="prepend-left-4"
- :link-href="commit.author.path"
- :img-src="commit.author.avatar_url"
+ :link-href="author.path"
+ :img-src="author.avatar_url"
:img-alt="userImageAltDescription"
- :tooltip-text="commit.author.username"
+ :tooltip-text="author.username"
/>
</div>
</div>
diff --git a/app/assets/javascripts/releases/index.js b/app/assets/javascripts/releases/index.js
index 6fa7298ac5a..adbed3cb8e2 100644
--- a/app/assets/javascripts/releases/index.js
+++ b/app/assets/javascripts/releases/index.js
@@ -14,7 +14,7 @@ export default () => {
render(createElement) {
return createElement('app', {
props: {
- endpoint: element.dataset.endpoint,
+ projectId: element.dataset.projectId,
documentationLink: element.dataset.documentationPath,
illustrationPath: element.dataset.illustrationPath,
},
diff --git a/spec/javascripts/releases/components/release_block_spec.js b/spec/javascripts/releases/components/release_block_spec.js
index 19aecbb3636..29420216bc4 100644
--- a/spec/javascripts/releases/components/release_block_spec.js
+++ b/spec/javascripts/releases/components/release_block_spec.js
@@ -15,6 +15,16 @@ describe('Release block', () => {
author_name: 'Release bot',
author_email: 'release-bot@example.com',
created_at: '2012-05-28T05:00:00-07:00',
+ author: {
+ avatar_url: 'uploads/-/system/user/avatar/johndoe/avatar.png',
+ id: 482476,
+ name: 'John Doe',
+ path: '/johndoe',
+ state: 'active',
+ status_tooltip_html: null,
+ username: 'johndoe',
+ web_url: 'https://gitlab.com/johndoe',
+ },
commit: {
id: '2695effb5807a22ff3d138d593fd856244e155e7',
short_id: '2695effb',
@@ -28,16 +38,6 @@ describe('Release block', () => {
committer_name: 'Jack Smith',
committer_email: 'jack@example.com',
committed_date: '2012-05-28T04:42:42-07:00',
- author: {
- avatar_url: 'uploads/-/system/user/avatar/johndoe/avatar.png',
- id: 482476,
- name: 'John Doe',
- path: '/johndoe',
- state: 'active',
- status_tooltip_html: null,
- username: 'johndoe',
- web_url: 'https://gitlab.com/johndoe',
- },
},
assets: {
count: 6,