https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10190 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #170 from Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> --- (In reply to Victor Grousset/tuxayo from comment #169)

Comment on attachment 178648 [details] [review] Bug 10190: Add a vue app for circulation triggers

Review of attachment 178648 [details] [review]: -----------------------------------------------------------------

::: koha-tmpl/intranet-tmpl/prog/en/modules/admin/circulation_triggers.tt @@ +25,5 @@

...

+ <!-- this is closed in intranet-bottom.inc --> + + [% MACRO jsinclude BLOCK %] + [% INCLUDE 'calendar.inc' %] + <!-- FIXME: this shouldn't be needed -->

Is the FIXME for calendar.inc above or datatables.inc bellow or the 4 includes below?

It was for calendar.inc and if you look at the other vue apps, it's consistent throughout them so far. -- You are receiving this mail because: You are watching all bug changes.