Some librarians would like the ability to leave messages attached to an item ( outside the scope of itemnotes and itemnotes_nonpublic ). This feature would give the librarians the ability to add and remove messages of arbitrary types defined in an authorised value list.
*** Bug 13733 has been marked as a duplicate of this bug. ***
Created attachment 55544 [details] [review] Bug 17297 - Add npm and webpack config files
Created attachment 55545 [details] [review] Bug 17297 [POC] - Add React based item message editor
Created attachment 55546 [details] [review] Bug 17297 [POC] - Add minified js file
This patch set is a proof of concept for using React for the item messages editor. It has no backend at the moment. It will be trivial to hook this widget into the REST API once the concept is approved. If you want to try it out: 1) Apply this patch set 2) Browse to moredetail.pl 3) Note each item has the same 5 pre-generated item messages 4) Test Adding and removing item messages If you want to hack on it: 1) Install npm 2) Apply this patch set 3) Run 'npm -i' to install development dependencies 4) Run 'webpack' to update the compiled moremember.js file to the dev version of React 5) Hack on it, run 'webpack' after making changes ( or look at the --watch option ) 6) Run webpack -p --config webpack-production.config.js to minify the moremember.js file before submitting an update to it
Created attachment 60861 [details] [review] [SIGNED-OFF] Bug 17297 - Add npm and webpack config files Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 60862 [details] [review] [SIGNED-OFF] Bug 17297 [POC] - Add React based item message editor Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 60863 [details] [review] [SIGNED-OFF] Bug 17297 [POC] - Add minified js file Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 60926 [details] [review] Bug 17297 - Add npm and webpack config files
Created attachment 60927 [details] [review] Bug 17297 - Add React based item message editor Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Created attachment 60928 [details] [review] Bug 17297 - Add minified js file
Just few things, qa tools complain: FAIL catalogue/moredetail.pl FAIL forbidden patterns forbidden pattern: Data::Dumper::Dumper (line 268) And the test plan does not mention the need of ITEM_MESSAGE_TYPE authorised values category. The input field get's smaller when I start to write a message. Maybe the trash icon would be better for removing messages... Do you have plan to show some messages summary information on normal detail page? Failed QA because of the usage of Data::Dumper
I think this was probably closed because of the React POC? Because I think the feature itself would still be very useful, I have re-filed a new enh. request: bug 27374
(In reply to Katrin Fischer from comment #13) > I think this was probably closed because of the React POC? Because I think > the feature itself would still be very useful, I have re-filed a new enh. > request: bug 27374 Both of those bugs patches are quite old at this point. We needed this feature post-haste and since we are still in the process of settling the react/vue debate, I created a plugin as a stop-gap: https://github.com/bywatersolutions/koha-plugin-item-messages We should consider this as a possibility for Koha, it is react based but requires no backend transpiling! In theory the same could be done for vue if that's what we go with.