Browse Source

Fix: more explicit (and correct) time entries delete URL

master
Ilya Konanykhin 7 years ago
parent
commit
76980a8903
  1. 2
      app/assets/javascripts/time_entries.js
  2. 2
      app/views/time_entries/index.haml

2
app/assets/javascripts/time_entries.js

@ -67,7 +67,7 @@ $(document).ready(function () {
$("#confirmation_delete").click(function () {
var entry_id = $(this).data("entry_id");
var url = $("#confirmation_delete").data("url") + entry_id;
var url = $("#confirmation_delete").data("url-template").replace(/__ID__/, entry_id);
$.ajax({
url: url,
type: "delete",

2
app/views/time_entries/index.haml

@ -41,7 +41,7 @@
#work_entry_description Description
.modal-footer
%button.btn.btn-default(data-dismiss="modal" aria-hidden="true") Cancel
%button.btn.btn-danger#confirmation_delete(data-url="api/v1/time_entries/") Delete
%button.btn.btn-danger#confirmation_delete{data: {url_template: api_delete_time_entry_path('__ID__')}} Delete

Loading…
Cancel
Save