Commit 28f33dcf authored by Natalia Tepluhina's avatar Natalia Tepluhina

Merge branch 'mrincon/remove-jquery-lodash-deps-api-js' into 'master'

Remove unnecessary noop dependencies

See merge request gitlab-org/gitlab!27291
parents 26dfc31b 95afc43f
import $ from 'jquery';
import _ from 'underscore';
import axios from './lib/utils/axios_utils'; import axios from './lib/utils/axios_utils';
import { joinPaths } from './lib/utils/url_utility'; import { joinPaths } from './lib/utils/url_utility';
import flash from '~/flash'; import flash from '~/flash';
...@@ -70,7 +68,7 @@ const Api = { ...@@ -70,7 +68,7 @@ const Api = {
}, },
// Return groups list. Filtered by query // Return groups list. Filtered by query
groups(query, options, callback = $.noop) { groups(query, options, callback = () => {}) {
const url = Api.buildUrl(Api.groupsPath); const url = Api.buildUrl(Api.groupsPath);
return axios return axios
.get(url, { .get(url, {
...@@ -108,7 +106,7 @@ const Api = { ...@@ -108,7 +106,7 @@ const Api = {
}, },
// Return projects list. Filtered by query // Return projects list. Filtered by query
projects(query, options, callback = _.noop) { projects(query, options, callback = () => {}) {
const url = Api.buildUrl(Api.projectsPath); const url = Api.buildUrl(Api.projectsPath);
const defaults = { const defaults = {
search: query, search: query,
......
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