On the inventory tool the submit button is a the very bottom, but it should probably be in two places. One under the file upload and one at the bottom. This will make it clear that if you do a file you don't have to fill in the filter options - just submit.
Created attachment 24705 [details] [review] [BUG 11272] Add second submit button to inventory The inventory tool allows for 2 ways of processing files. The first is to upload a file. The second is to generate the shelf list. Most libraries think they have to fill in all fields because the submit button is at the bottom of the second option. They do not. This patch adds a second submit button under the first method to make this clearer. To test: * Generate a file of barcodes for inventorying * Before applying the patch use the file upload method to mark as seen * Apply the patch * Use the file upload method to mark as seen using the new submit * Repeat the above for the shelf list method * Confirm that inventory still works as expected with 2 buttons
Created attachment 24893 [details] [review] [SIGNED-OFF] Bug 11272 - copy submit button on inventory The inventory tool allows for 2 ways of processing files. The first is to upload a file. The second is to generate the shelf list. Most libraries think they have to fill in all fields because the submit button is at the bottom of the second option. They do not. This patch adds a second submit button under the first method to make this clearer. To test: * Generate a file of barcodes for inventorying * Before applying the patch use the file upload method to mark as seen * Apply the patch * Use the file upload method to mark as seen using the new submit * Repeat the above for the shelf list method * Confirm that inventory still works as expected with 2 buttons Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 24934 [details] [review] Bug 11272 - copy submit button on inventory The inventory tool allows for 2 ways of processing files. The first is to upload a file. The second is to generate the shelf list. Most libraries think they have to fill in all fields because the submit button is at the bottom of the second option. They do not. This patch adds a second submit button under the first method to make this clearer. To test: * Generate a file of barcodes for inventorying * Before applying the patch use the file upload method to mark as seen * Apply the patch * Use the file upload method to mark as seen using the new submit * Repeat the above for the shelf list method * Confirm that inventory still works as expected with 2 buttons Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Passing QA on this trivial patch adding a submit button on inventory, but I feel that this form actually needs more attention as for design. Just viewing this as a workaround. The form design is not intuitive. The two approaches should be more visible, as well as common options to both approaches like exporting to csv etc.
Pushed to master. Thanks, Nicole! I agree with Marcel that the form needs a revamp to make it clear that it has two distinct modes of operation.
Pushed to 3.14.x, will be in 3.14.07
Pushed to 3.12.x will be in 3.12.13
I'm wondering if this got reverted. I'm on 3.16 (and I have a master system as well) and I don't see the button anymore. Can anyone confirm? Nicole
Bug 12025 as reverted this bug
Created attachment 37850 [details] [review] Bug 11272 - copy submit button on inventory The inventory tool allows for 2 ways of processing files. The first is to upload a file. The second is to generate the shelf list. Most libraries think they have to fill in all fields because the submit button is at the bottom of the second option. They do not. This patch adds a second submit button under the first method to make this clearer. To test: * Generate a file of barcodes for inventorying * Before applying the patch use the file upload method to mark as seen * Apply the patch * Use the file upload method to mark as seen using the new submit * Repeat the above for the shelf list method * Confirm that inventory still works as expected with 2 buttons Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Rebased.
I wasn't sure what status to stick this to. If you compare it to the previous obsolete, it's identical. For some strange reason the extra blank lines confused the git bz apply such that it didn't apply. This one does.
Created attachment 37895 [details] [review] [Signed-off] Bug 11272 - copy submit button on inventory The inventory tool allows for 2 ways of processing files. The first is to upload a file. The second is to generate the shelf list. Most libraries think they have to fill in all fields because the submit button is at the bottom of the second option. They do not. This patch adds a second submit button under the first method to make this clearer. To test: * Generate a file of barcodes for inventorying * Before applying the patch use the file upload method to mark as seen * Apply the patch * Use the file upload method to mark as seen using the new submit * Repeat the above for the shelf list method * Confirm that inventory still works as expected with 2 buttons Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Rebased. Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Mark, please have a look at the historic of the report before reintroduce a dead patch. It will take you to bug 12025 comment 4.
This is "resolved" as it is pushed to master and stable versions. Open a new bug for discussion if needed.
(In reply to Jonathan Druart from comment #13) > Mark, please have a look at the historic of the report before reintroduce a > dead patch. It will take you to bug 12025 comment 4. Apologies. I was just working at lowering the "Patch does not apply" queue. I should have been a little more thorough in checking git logs, and noting see also's above.