diff options
author | Robert Speicher <robert@gitlab.com> | 2016-03-15 03:51:43 +0300 |
---|---|---|
committer | Robert Speicher <robert@gitlab.com> | 2016-03-15 03:51:43 +0300 |
commit | c4b35a623016bb8a3402d9a53fb9df7a990ad9aa (patch) | |
tree | 2f5f0dee79ce97c64e2d3fc37d166f3152484033 /lib | |
parent | c1d7e5ac1818830127ca7b10fcb7c862e512bfae (diff) | |
parent | 16592e2b45d42e22f9d1d595a1f44821c7b30441 (diff) |
Merge branch 'fix-commit-status-rendering' into 'master'
Cleanup Ci::Commit, Ci::Build and CommitStatus views
This MR tries to do first sweep of cleanups to Ci::Commit and
Ci::Build objects removing all view-related functions and fixing the
API from other side.
Fixes: https://gitlab.com/gitlab-org/gitlab-ce/issues/6046
See merge request !2760
Diffstat (limited to 'lib')
-rw-r--r-- | lib/api/entities.rb | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/lib/api/entities.rb b/lib/api/entities.rb index 7204dca34ba..9805e53624e 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -408,13 +408,6 @@ module API expose :id, :status, :stage, :name, :ref, :tag, :coverage expose :created_at, :started_at, :finished_at expose :user, with: User - # TODO: download_url in Ci:Build model is an GitLab Web Interface URL, not API URL. We should think on some API - # for downloading of artifacts (see: https://gitlab.com/gitlab-org/gitlab-ce/issues/4255) - expose :download_url do |repo_obj, options| - if options[:user_can_download_artifacts] - repo_obj.artifacts_download_url - end - end expose :artifacts_file, using: BuildArtifactFile, if: -> (build, opts) { build.artifacts? } expose :commit, with: RepoCommit do |repo_obj, _options| if repo_obj.respond_to?(:commit) |