django-check-seo icon indicating copy to clipboard operation
django-check-seo copied to clipboard

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

Open jgadelange opened this issue 2 years ago • 3 comments

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.

jgadelange avatar Sep 13 '22 12:09 jgadelange

Hi!

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

corentinbettiol avatar Jan 25 '24 12:01 corentinbettiol

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

jgadelange avatar Jan 25 '24 13:01 jgadelange

True.

corentinbettiol avatar Jan 25 '24 13:01 corentinbettiol