Merge branch '63474-applying-a-quick-action-does-not-show-the-correct-message' into 'master'

Fixed bug from extra parenthesis

Closes #63474

See merge request gitlab-org/gitlab-ce!31479
This commit is contained in:
Clement Ho 2019-08-05 18:49:56 +00:00
commit b683d6e65f
2 changed files with 7 additions and 5 deletions

View File

@ -21,7 +21,7 @@ export const getQuickActionText = note => {
text = __('Applying multiple commands');
} else {
const commandDescription = executedCommands[0].description.toLowerCase();
text = sprintf(__('Applying command to %{commandDescription}', { commandDescription }));
text = sprintf(__('Applying command to %{commandDescription}'), { commandDescription });
}
}

View File

@ -101,10 +101,12 @@ Sidebar.prototype.toggleTodo = function(e) {
this.todoUpdateDone(data);
})
.catch(() =>
flash(sprintf(__('There was an error %{message} todo.')), {
message:
ajaxType === 'post' ? s__('RightSidebar|adding a') : s__('RightSidebar|deleting the'),
}),
flash(
sprintf(__('There was an error %{message} todo.'), {
message:
ajaxType === 'post' ? s__('RightSidebar|adding a') : s__('RightSidebar|deleting the'),
}),
),
);
};