search

kapt-labs / django-check-seo

Django Check SEO will check the SEO aspects of your site for you, and will provide advice in case of problems. Compatible with Django & Django-CMS!
GNU General Public License v3.0
154 stars 12 forks source link

Enable/disable django-check-seo on a per user basis #53

Open jgadelange opened 1 year ago

jgadelange commented 1 year ago

Is your feature request related to a problem? Please describe. For some of our clients we would like to be able to disable django-check-seo for them, but have it available to ourselves.

Describe the solution you'd like I would like to add a permission in the default django user admin that we can give to the users/groups that should be able to view the django-check-seo page. I believe the current behaviour should also be supported (only staff users can view the page), therefore I think we can add a setting (e.g. DJANGO_CHECK_SEO_USE_PERMISSIONS) that toggles the behaviour.

Describe alternatives you've considered N/A

Additional context In the future we probably also want be able to have access control on a per check basis (so not show all checks to every user), so that could be something that could be considered while implementing this feature.

corentinbettiol commented 7 months ago

Hi!

I added a new permission in django check seo 1.0.0 :)

jgadelange commented 7 months ago

I think we should should also remove django-check-seo from the cms menu: https://github.com/kapt-labs/django-check-seo/blob/master/django_check_seo/cms_toolbars.py#L16

corentinbettiol commented 7 months ago

True.