Skip to content
Snippets Groups Projects
Commit 5baea769 authored by Piotr Gawron's avatar Piotr Gawron
Browse files

Merge branch '433-removing-comments-doesn-t-work' into 'master'

Resolve "removing comments doesn't work"

Closes #433

See merge request !327
parents 7f1472b1 ed48a9b8
No related branches found
No related tags found
1 merge request!327Resolve "removing comments doesn't work"
Pipeline #
......@@ -1833,6 +1833,15 @@ ServerConnector.removeOverlay = function (params) {
});
};
/**
*
* @param {Object} params
* @param {number} params.commentId
* @param {string} [params.reason]
* @param {string} [params.projectId]
*
* @returns {Promise}
*/
ServerConnector.removeComment = function (params) {
var self = this;
var queryParams = {
......
......@@ -3,6 +3,7 @@
var AbstractGuiElement = require('../AbstractGuiElement');
var Functions = require('../../Functions');
var GuiConnector = require('../../GuiConnector');
// noinspection JSUnusedLocalSymbols
var logger = require('../../logger');
......@@ -78,13 +79,14 @@ CommentsAdminPanel.prototype.refreshComments = function () {
if (param.status) {
return ServerConnector.removeComment({
commentId: $(button).attr("data"),
reason: param.reason
reason: param.reason,
projectId: self.getProject().getProjectId()
}).then(function () {
$(button).after("<span>YES (" + param.reason + ")</span>");
button.style.display = "none";
});
}
})
}).catch(GuiConnector.alert)
});
});
};
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment