Closed (fixed)
Project:
Link checker
Version:
2.0.x-dev
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Nov 2023 at 11:49 UTC
Updated:
19 Aug 2024 at 07:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
boulaffasae commentedComment #3
boulaffasae commentedI also added view permission, it should be a custom permission instead of true by default :)
Comment #4
eiriksmAwesome 👍 Thanks for the patch!
Would be nice to have:
- The changes in a merge request
- some tests for this
Comment #7
quadrexdevAdded a merge request with the changes from #3 + added a simple Kernel test to verify it's working.
Please review
Comment #8
quadrexdevOops, changed status to needs review before tests passed so reverting it back
Comment #9
quadrexdevOk, now it's passing.
So after applying a patch from #3 all the users had access to view the linkchecker entity (and that was the issue why LinkCheckerLinkAccessTest was failing). I reverted this part cuz the viewing operation is not supposed to be allowed for all the users, right?
Please review
Comment #11
quadrexdevMerged in 2.0.x