Closed mshannaq closed 11 months ago
I'd like to propose a change for the Controllers/Url.php
controller. Instead of using queries directly within the controller, I suggest utilizing the getUrlsForUser()
function and other functions I plan to create later. This approach will enhance code organization and maintainability.
I'd like to propose a change for the
Controllers/Url.php
controller. Instead of using queries directly within the controller, I suggest utilizing thegetUrlsForUser()
function and other functions I plan to create later. This approach will enhance code organization and maintainability.
done in commit 5ba50fc356e3a8246c4d1a4bcc9de36a41c8bd11
Pull Request Summary:
In this PR, I am completing the ListURLs layout with the correct view and data.
Changes:
Checklist:
url.access
only and add 'All URLs' to the sidebar for users with appropriate permissions. and make sure that the default URL list isurl/user/{user_id}
Please review and merge as needed. Thank you!