Update API on frontend to use v4

Use options object to pass params for project endpoint
This commit is contained in:
Sam Rose 2017-03-06 00:03:21 -05:00
parent b3eda94445
commit 435458d2b1
6 changed files with 11 additions and 11 deletions

View File

@ -51,15 +51,15 @@ var Api = {
}); });
}, },
// Return projects list. Filtered by query // Return projects list. Filtered by query
projects: function(query, order, callback) { projects: function(query, options, callback) {
var url = Api.buildUrl(Api.projectsPath); var url = Api.buildUrl(Api.projectsPath);
return $.ajax({ return $.ajax({
url: url, url: url,
data: { data: $.extend({
search: query, search: query,
order_by: order, per_page: 20,
per_page: 20 membership: true
}, }, options),
dataType: "json" dataType: "json"
}).done(function(projects) { }).done(function(projects) {
return callback(projects); return callback(projects);

View File

@ -35,7 +35,7 @@
if (this.groupId) { if (this.groupId) {
return Api.groupProjects(this.groupId, term, projectsCallback); return Api.groupProjects(this.groupId, term, projectsCallback);
} else { } else {
return Api.projects(term, orderBy, projectsCallback); return Api.projects(term, { order_by: orderBy }, projectsCallback);
} }
}, },
url: function(project) { url: function(project) {
@ -84,7 +84,7 @@
if (_this.groupId) { if (_this.groupId) {
return Api.groupProjects(_this.groupId, query.term, projectsCallback); return Api.groupProjects(_this.groupId, query.term, projectsCallback);
} else { } else {
return Api.projects(query.term, _this.orderBy, projectsCallback); return Api.projects(query.term, { order_by: _this.orderBy }, projectsCallback);
} }
}; };
})(this), })(this),

View File

@ -47,7 +47,7 @@
fields: ['name'] fields: ['name']
}, },
data: function(term, callback) { data: function(term, callback) {
return Api.projects(term, 'id', function(data) { return Api.projects(term, { order_by: 'id' }, function(data) {
data.unshift({ data.unshift({
name_with_namespace: 'Any' name_with_namespace: 'Any'
}); });

View File

@ -410,7 +410,7 @@
:javascript :javascript
$('#js-project-dropdown').glDropdown({ $('#js-project-dropdown').glDropdown({
data: function (term, callback) { data: function (term, callback) {
Api.projects(term, "last_activity_at", function (data) { Api.projects(term, { order_by: 'last_activity_at' }, function (data) {
callback(data); callback(data);
}); });
}, },

View File

@ -1,7 +1,7 @@
module Gitlab module Gitlab
module GonHelper module GonHelper
def add_gon_variables def add_gon_variables
gon.api_version = 'v3' # v4 Is not officially released yet, therefore can't be considered as "frozen" gon.api_version = 'v4'
gon.default_avatar_url = URI.join(Gitlab.config.gitlab.url, ActionController::Base.helpers.image_path('no_avatar.png')).to_s gon.default_avatar_url = URI.join(Gitlab.config.gitlab.url, ActionController::Base.helpers.image_path('no_avatar.png')).to_s
gon.max_file_size = current_application_settings.max_attachment_size gon.max_file_size = current_application_settings.max_attachment_size
gon.asset_host = ActionController::Base.asset_host gon.asset_host = ActionController::Base.asset_host

View File

@ -26,7 +26,7 @@ require('~/project');
var fakeAjaxResponse = function fakeAjaxResponse(req) { var fakeAjaxResponse = function fakeAjaxResponse(req) {
var d; var d;
expect(req.url).toBe('/api/v3/projects.json?simple=true'); expect(req.url).toBe('/api/v3/projects.json?simple=true');
expect(req.data).toEqual({ search: '', order_by: 'last_activity_at', per_page: 20 }); expect(req.data).toEqual({ search: '', order_by: 'last_activity_at', per_page: 20, membership: true });
d = $.Deferred(); d = $.Deferred();
d.resolve(this.projects_data); d.resolve(this.projects_data);
return d.promise(); return d.promise();