[Bug 33363] New: More specific permissions for purchase suggestions
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Bug ID: 33363 Summary: More specific permissions for purchase suggestions Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: enhancement Priority: P5 - low Component: Acquisitions Assignee: koha-bugs@lists.koha-community.org Reporter: bwsdonna@gmail.com QA Contact: testopia@bugs.koha-community.org There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Currently if staff are given Suggestions Management/Manage Purchase Suggestions, and no Acq permissions, they are able to not only submit/view from the borrower record, but also access the purchase suggestion module and update/approve/delete purchase suggestions. If Suggestions Management is not turned on, then staff cannot enter a purchase suggestion from a borrower record, nor can they see them from the patron record to let the patron know what the status is. It would be helpful to have an addition permission level that would allow staff to enter a purchase suggestion from the borrower record, and view the existing purchase suggestions on a patron account. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kristi <kkrueger@cuyahogalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kkrueger@cuyahogalibrary.or | |g -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #1 from Kristi <kkrueger@cuyahogalibrary.org> --- Thank you for submitting a bug report for this, Donna! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Daniel B. <dbarden@cuyahogalibrary.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dbarden@cuyahogalibrary.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Jessie Zairo <jzairo@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jzairo@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kyle@bywatersolutions.com Assignee|koha-bugs@lists.koha-commun |kyle@bywatersolutions.com |ity.org | -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #2 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 150247 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150247&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 PTFS Europe Sandboxes <sandboxes@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #150247|0 |1 is obsolete| | --- Comment #3 from PTFS Europe Sandboxes <sandboxes@ptfs-europe.com> --- Created attachment 150450 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=150450&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Michaela Sieber <michaela.sieber@kit.edu> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |michaela.sieber@kit.edu -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Marcel de Rooy <m.de.rooy@rijksmuseum.nl> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply CC| |m.de.rooy@rijksmuseum.nl -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #150450|0 |1 is obsolete| | --- Comment #4 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 153177 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=153177&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Joseph Alway <jbalway@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jbalway@gmail.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katie Bliss <kebliss@dmpl.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kebliss@dmpl.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Patch doesn't apply CC| |jonathan.druart+koha@gmail. | |com --- Comment #5 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- Please rebase and add REST API tests. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kris <kbecker@jcls.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |kbecker@jcls.org -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #153177|0 |1 is obsolete| | --- Comment #6 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 160846 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160846&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 160847 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160847&action=edit Bug 33363: Add unit tests -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lisette.scheer@bywatersolut | |ions.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |martin.renvoize@ptfs-europe |y.org |.com CC| |martin.renvoize@ptfs-europe | |.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #160846|0 |1 is obsolete| | --- Comment #8 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 160902 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160902&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #160847|0 |1 is obsolete| | --- Comment #9 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 160903 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=160903&action=edit Bug 33363: Add unit tests Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #10 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Tidied and squash said tidies. QA scripts now happy, tests make sense and are passing.. controllers updated appropriately. Passing QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Patch doesn't apply --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am sorry, this no longer applies. Can I please get a rebase? It will jump right up back to the front of the list after. Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 33363: Add separate permissions for creating and deleting suggestions Using index info to reconstruct a base tree... M installer/data/mysql/mandatory/userpermissions.sql M koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc M koha-tmpl/intranet-tmpl/prog/en/includes/header.inc M koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc M koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc M koha-tmpl/intranet-tmpl/prog/en/modules/members/purchase-suggestions.tt M koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt M members/purchase-suggestions.pl M suggestion/suggestion.pl M t/Koha/Auth/Permissions.t Falling back to patching base and 3-way merge... Auto-merging t/Koha/Auth/Permissions.t Auto-merging suggestion/suggestion.pl CONFLICT (content): Merge conflict in suggestion/suggestion.pl Auto-merging members/purchase-suggestions.pl Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/purchase-suggestions.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/permissions.inc Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/header.inc Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc Auto-merging installer/data/mysql/mandatory/userpermissions.sql error: Failed to merge in the changes. Patch failed at 0001 Bug 33363: Add separate permissions for creating and deleting suggestions hint: Use 'git am --show-current-patch=diff' to see the failed patch When you have resolved this problem run "git bz apply --continue". If you would prefer to skip this patch, instead run "git bz apply --skip". To restore the original branch and stop patching run "git bz apply --abort". Patch left in /tmp/Bug-33363-Add-separate-permissions-for-creating-an-4fjafpom.patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #160902|0 |1 is obsolete| | Attachment #160903|0 |1 is obsolete| | --- Comment #12 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 165142 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=165142&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #13 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 165143 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=165143&action=edit Bug 33363: Add unit tests -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #165142|0 |1 is obsolete| | --- Comment #14 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 168052 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168052&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #165143|0 |1 is obsolete| | --- Comment #15 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- Created attachment 168053 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168053&action=edit Bug 33363: Add unit tests Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #16 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- That was a fun rebase.. back to PQA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 ByWater Sandboxes <bws.sandboxes@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #168052|0 |1 is obsolete| | --- Comment #17 from ByWater Sandboxes <bws.sandboxes@gmail.com> --- Created attachment 168055 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168055&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 ByWater Sandboxes <bws.sandboxes@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #168053|0 |1 is obsolete| | --- Comment #18 from ByWater Sandboxes <bws.sandboxes@gmail.com> --- Created attachment 168056 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168056&action=edit Bug 33363: Add unit tests Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Failed QA --- Comment #19 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Some small stuff, please follow-up: 1) Database update It's not using say_success, but it's also not giving us any success message :) 2) What is actually meant by "manage"? I assume that "manage" actually means "edit". While any of the permissions allow to "view". If that is the case, I feel it would be good to adjust the permission descriptions accordingly. There is a slight discrepancy tho: With only create permission, I have the entry to suggestions in the "more" menu, but it's missing from the navigation in the acquisition module. I can also not see the pending suggestions on the start page. I think we should adjust the navigation in the acq module at least. Should the pending suggestions on acq and staff interface home page be visible with manage_suggestions only or also with any suggestion permission? 3) Question + <div class="note">Only staff with superlibrarian or full suggestions permissions are returned in the search results</div> I think this is in line with the previous behavior, but I wonder if we should not change to allow "any" instead of "full" here. I assume this is for setting the manager of a suggestion - editing might be required, but not sure if delete and create would be as well. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #20 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Katrin Fischer from comment #19)
Some small stuff, please follow-up:
1) Database update
It's not using say_success, but it's also not giving us any success message :)
Done
2) What is actually meant by "manage"?
I assume that "manage" actually means "edit". While any of the permissions allow to "view".
If that is the case, I feel it would be good to adjust the permission descriptions accordingly.
That was the pre-existing term and in Koha we've always used it as a catch-all for anything not covered by more specific permissions. It covers more than just viewing, such as approvals.
There is a slight discrepancy tho: With only create permission, I have the entry to suggestions in the "more" menu, but it's missing from the navigation in the acquisition module.
Can you add a screenshot or be more specific about where this pulldown is?
I can also not see the pending suggestions on the start page.
That is intended. A librarian that can only create suggestions has no need to see the suggestions that need managed.
I think we should adjust the navigation in the acq module at least.
Should the pending suggestions on acq and staff interface home page be visible with manage_suggestions only or also with any suggestion permission?
They should be visible only for manage_suggestions.
3) Question
+ <div class="note">Only staff with superlibrarian or full suggestions permissions are returned in the search results</div>
I think this is in line with the previous behavior, but I wonder if we should not change to allow "any" instead of "full" here.
I disagree. This is still a limit for just librarians that manage permissions, just like all the other manage permissions we can limit patron searches too.
I assume this is for setting the manager of a suggestion - editing might be required, but not sure if delete and create would be as well.
They would not be. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #168055|0 |1 is obsolete| | Attachment #168056|0 |1 is obsolete| | --- Comment #21 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168356 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168356&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #22 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168357 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168357&action=edit Bug 33363: Add unit tests Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #23 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168358 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168358&action=edit Bug 33363: (QA Follow-up) Use say_success in atomic update -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #24 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168359 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168359&action=edit Bug 33363: (QA Follow-up) Fix bug preventing Suggestions from showing in Acquistions menu -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #25 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Kyle, some quick explanations to keep this going: (In reply to Kyle M Hall from comment #20)
(In reply to Katrin Fischer from comment #19)
Some small stuff, please follow-up:
1) Database update
It's not using say_success, but it's also not giving us any success message :)
Done
Thanks!
2) What is actually meant by "manage"?
I assume that "manage" actually means "edit". While any of the permissions allow to "view".
If that is the case, I feel it would be good to adjust the permission descriptions accordingly.
That was the pre-existing term and in Koha we've always used it as a catch-all for anything not covered by more specific permissions. It covers more than just viewing, such as approvals.
We have used it as a catch all, I agree, it meant: CRUD. But now that we are starting to split permissions up, I think we should try to be as precise in our descriptions as possible. For example: manage so far included deleting, but it does no longer. What do you think about "View and edit ..." as a description to replace "Manage..." (not renaming permission code, but permission description)
There is a slight discrepancy tho: With only create permission, I have the entry to suggestions in the "more" menu, but it's missing from the navigation in the acquisition module.
Can you add a screenshot or be more specific about where this pulldown is?
The "More" in the main navigation of Koha on top of every page :) I think it should probably be visible with "any" suggestion permission, what do you think? The same for the acquisition module navigation on the left.
I can also not see the pending suggestions on the start page.
That is intended. A librarian that can only create suggestions has no need to see the suggestions that need managed.
I think we should adjust the navigation in the acq module at least.
see above
Should the pending suggestions on acq and staff interface home page be visible with manage_suggestions only or also with any suggestion permission?
They should be visible only for manage_suggestions.
OK, so we leave the pending suggestions display for the "manage" permission.
3) Question
+ <div class="note">Only staff with superlibrarian or full suggestions permissions are returned in the search results</div>
I think this is in line with the previous behavior, but I wonder if we should not change to allow "any" instead of "full" here.
I disagree. This is still a limit for just librarians that manage permissions, just like all the other manage permissions we can limit patron searches too.
I assume this is for setting the manager of a suggestion - editing might be required, but not sure if delete and create would be as well.
They would not be.
But if the search is limited to must have "all module" permissions, I cannot edit someone that has only manage or only delete or only create to be a manager. I think the "only manage" could maybe be a use case? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off --- Comment #26 from Kyle M Hall <kyle@bywatersolutions.com> ---
2) What is actually meant by "manage"?
I assume that "manage" actually means "edit". While any of the permissions allow to "view".
If that is the case, I feel it would be good to adjust the permission descriptions accordingly.
That was the pre-existing term and in Koha we've always used it as a catch-all for anything not covered by more specific permissions. It covers more than just viewing, such as approvals.
We have used it as a catch all, I agree, it meant: CRUD. But now that we are starting to split permissions up, I think we should try to be as precise in our descriptions as possible.
I disagree with that interpretation, but I'm open to other possibilities. I think we've hit something philosophical that it far outside the scope of this development. Maybe we should discuss this at the next dev meeting?
Can you add a screenshot or be more specific about where this pulldown is?
The "More" in the main navigation of Koha on top of every page :)
I think it should probably be visible with "any" suggestion permission, what do you think?
The same for the acquisition module navigation on the left.
You found a bug! Fixed!
3) Question
+ <div class="note">Only staff with superlibrarian or full suggestions permissions are returned in the search results</div>
I think this is in line with the previous behavior, but I wonder if we should not change to allow "any" instead of "full" here.
I disagree. This is still a limit for just librarians that manage permissions, just like all the other manage permissions we can limit patron searches too.
I assume this is for setting the manager of a suggestion - editing might be required, but not sure if delete and create would be as well.
They would not be.
But if the search is limited to must have "all module" permissions, I cannot edit someone that has only manage or only delete or only create to be a manager. I think the "only manage" could maybe be a use case?
I'm not sure I understand. Those searches are already filtering for the "suggestions_manage". Those searches are for setting a manager for the suggestion. Everything works as is, right? I did not make any changes to this code. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #27 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Kyle M Hall from comment #26)
2) What is actually meant by "manage"?
I assume that "manage" actually means "edit". While any of the permissions allow to "view".
If that is the case, I feel it would be good to adjust the permission descriptions accordingly.
That was the pre-existing term and in Koha we've always used it as a catch-all for anything not covered by more specific permissions. It covers more than just viewing, such as approvals.
We have used it as a catch all, I agree, it meant: CRUD. But now that we are starting to split permissions up, I think we should try to be as precise in our descriptions as possible.
I disagree with that interpretation, but I'm open to other possibilities. I think we've hit something philosophical that it far outside the scope of this development. Maybe we should discuss this at the next dev meeting?
Maybe I am misunderstanding... but with manage_suggestions I can no longer delete/create now. So it feels absolutely necessary to me to adapt the permission description?
Can you add a screenshot or be more specific about where this pulldown is?
The "More" in the main navigation of Koha on top of every page :)
I think it should probably be visible with "any" suggestion permission, what do you think?
The same for the acquisition module navigation on the left.
You found a bug! Fixed!
3) Question
+ <div class="note">Only staff with superlibrarian or full suggestions permissions are returned in the search results</div>
I think this is in line with the previous behavior, but I wonder if we should not change to allow "any" instead of "full" here.
I disagree. This is still a limit for just librarians that manage permissions, just like all the other manage permissions we can limit patron searches too.
I assume this is for setting the manager of a suggestion - editing might be required, but not sure if delete and create would be as well.
They would not be.
But if the search is limited to must have "all module" permissions, I cannot edit someone that has only manage or only delete or only create to be a manager. I think the "only manage" could maybe be a use case?
I'm not sure I understand. Those searches are already filtering for the "suggestions_manage". Those searches are for setting a manager for the suggestion. Everything works as is, right? I did not make any changes to this code.
I was referring to these lines here: - <div class="note">Only staff with superlibrarian or suggestions_manage permissions are returned in the search results</div> + <div class="note">Only staff with superlibrarian or full suggestions permissions are returned in the search results</div> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #28 from Katie Bliss <kebliss@dmpl.org> --- Glad to see this one seems to be moving along! We get about 250-300 suggestions a month, so we're very interested in bugs like this. Question about the staff with create-only permissions: when you say that the intention is for those staff to NOT be able to view pending suggestions, does that mean they can't see them anywhere? Would they be able to see them on a patron's account? The comments above seem to primarily be about the link on the homepage that lets you know there are pending suggestions to be managed, but I wasn't sure. For us, we have two staff who are in charge of managing/making decisions on purchase suggestions, while pretty much everyone else in our system would only need the ability to create them. That said, the "everyone else" are our front-line staff who need to be able to provide information to our patrons, such as the status of their suggestions. So, it would make sense that they would be able to see a patron's suggestions, regardless of status, even if they can't edit/delete them. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #29 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Katie Bliss from comment #28)
Glad to see this one seems to be moving along! We get about 250-300 suggestions a month, so we're very interested in bugs like this.
Question about the staff with create-only permissions: when you say that the intention is for those staff to NOT be able to view pending suggestions, does that mean they can't see them anywhere? Would they be able to see them on a patron's account? The comments above seem to primarily be about the link on the homepage that lets you know there are pending suggestions to be managed, but I wasn't sure.
The create and delete permissions 'include' the view permission. I hope it came across that I have nothing against the feature - but I'd like us to really clearly define what each permission does/includes and describe the permissions accordingly. I think this will help to avoid misunderstandings and inconsistencies along the line when we are going to split up other manage permissions as well.
For us, we have two staff who are in charge of managing/making decisions on purchase suggestions, while pretty much everyone else in our system would only need the ability to create them. That said, the "everyone else" are our front-line staff who need to be able to provide information to our patrons, such as the status of their suggestions. So, it would make sense that they would be able to see a patron's suggestions, regardless of status, even if they can't edit/delete them.
The purchase suggestions tab in the patron account in the staff interface will be accessible with any of the suggestions permissions (delete, create and manage/edit permission) with this patch set. + flagsrequired => { suggestions => '*' }, -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #30 from Katie Bliss <kebliss@dmpl.org> --- Thank you for clarifying! I just wanted to make sure I understood what you and Kyle were discussing. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #31 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Katie Bliss from comment #30)
Thank you for clarifying! I just wanted to make sure I understood what you and Kyle were discussing.
Thanks for asking! I hope the explanation made sense! I have also added this to the agenda for the meeting tomorrow as Kyle suggested, so maybe we can move this forward there: https://wiki.koha-community.org/wiki/Development_IRC_meeting_3_July_2024 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #32 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- What's the state of this right now? It was PQA but is now back to SO... -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #33 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Another small follow-up is needed to update the permission descriptions as discussed at the dev meeting. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #34 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168803 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168803&action=edit Bug 33363: (QA Follow-up) Update manage permission description -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #168356|0 |1 is obsolete| | Attachment #168357|0 |1 is obsolete| | Attachment #168358|0 |1 is obsolete| | Attachment #168359|0 |1 is obsolete| | Attachment #168803|0 |1 is obsolete| | --- Comment #35 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168804 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168804&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #36 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168805 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168805&action=edit Bug 33363: Add unit tests Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #37 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168806 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168806&action=edit Bug 33363: (QA Follow-up) Use say_success in atomic update -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #38 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168807 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168807&action=edit Bug 33363: (QA Follow-up) Fix bug preventing Suggestions from showing in Acquistions menu -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #39 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168808 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168808&action=edit Bug 33363: (QA Follow-up) Update manage permission description -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #40 from Kyle M Hall <kyle@bywatersolutions.com> --- (In reply to Katrin Fischer from comment #33)
Another small follow-up is needed to update the permission descriptions as discussed at the dev meeting.
Katrin, did I get you request right? 🤞 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #41 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- (In reply to Kyle M Hall from comment #40)
(In reply to Katrin Fischer from comment #33)
Another small follow-up is needed to update the permission descriptions as discussed at the dev meeting.
Katrin, did I get you request right? 🤞
Yep, I think we can go with that :) Do you want another sign-off? -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #168804|0 |1 is obsolete| | Attachment #168805|0 |1 is obsolete| | Attachment #168806|0 |1 is obsolete| | Attachment #168807|0 |1 is obsolete| | Attachment #168808|0 |1 is obsolete| | --- Comment #42 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168865 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168865&action=edit Bug 33363: Add separate permissions for creating and deleting suggestions There are a number of libraries that would like for their staff to be able to submit (and view existing) purchase suggestions from the borrower record, but not give the staff the ability to edit/manage/delete purchase suggestions. Test Plan: 1) Apply this patch 2) Run restart all the things! 3) Run updatedatabase 4) Verify anyone with the suggestions manage permissions now has the create and delete permissions as well 5) Verify anyone without the suggestions manage permission has not recieved the new permissions 6) Enable only the create permission for a librarian 7) Verify that librarian can only create new suggestions ( not manage or delete ) 8) Enable only the manage permission for a librarian 9) Verify that librarian can only manage existing suggestions ( not create or delete ) 10) Enable only the delete permission for a librarian 11) Verify that librarian can only delete suggestions ( not create or manage ) Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Sponsored-by: Cuyahoga County Public Library -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #43 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168866 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168866&action=edit Bug 33363: Add unit tests Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Signed-off-by: Michaela Sieber <michaela.sieber@kit.edu> Sponsored-by: Cuyahoga County Public Library -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #44 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168867 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168867&action=edit Bug 33363: (QA Follow-up) Use say_success in atomic update -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #45 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168868 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168868&action=edit Bug 33363: (QA Follow-up) Fix bug preventing Suggestions from showing in Acquistions menu -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #46 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 168869 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=168869&action=edit Bug 33363: (QA Follow-up) Update manage permission description -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA QA Contact|martin.renvoize@ptfs-europe |katrin.fischer@bsz-bw.de |.com | --- Comment #47 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Last changes were just text and by my request, we got another sign-off - picking this out of the QA queue. Fixed inline: Processing additional checks * Commit title does not contain 'follow-up' correctly spelt - 66da72c23e4 * Commit title does not contain '(QA follow-up)' correctly spelt - 66da72c23e4 * Commit title does not contain 'follow-up' correctly spelt - f93263600af * Commit title does not contain '(QA follow-up)' correctly spelt - f93263600af * Commit title does not contain 'follow-up' correctly spelt - 42bd595ac80 * Commit title does not contain '(QA follow-up)' correctly spelt - 42bd595ac80 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to main Version(s)| |24.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 --- Comment #48 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Pushed for 24.11! Well done everyone, thank you! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Martin Renvoize (ashimema) <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |release-notes-needed -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com Status|Pushed to main |Needs documenting -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This enhancement adds new release notes| |staff permissions for | |suggestions. The new | |permissions are | |suggestions_create and | |suggestions_delete. Staff | |that currently have | |suggestions_manage will | |have the new statuses after | |the update. Keywords|release-notes-needed | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Documentation| |https://gitlab.com/koha-com submission| |munity/koha-manual/-/merge_ | |requests/1106 Documentation| |Caroline Cyr La Rose contact| | Status|Needs documenting |RESOLVED Resolution|--- |FIXED CC| |caroline.cyr-la-rose@inlibr | |o.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |emmi.takkinen@koha-suomi.fi --- Comment #49 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- There is a typo in atomicupdate file bug_33363.pl, permissions are now added to all users with full catalogue permissions. Filed a bug 41045. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33363 Jonathan Druart <jonathan.druart@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |41045 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41045 [Bug 41045] Suggestions manage permissions added to patrons who previously had no permissions in that category -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org