Marta Miranda / boletin_api

Added 3 commits:

• bb3b1e05...7b51bfd3 - 2 commits from branch develop
• b7018167 - Solucionar conflictos base.txt

Por qué haces una implementación Custom de DjangoModelPermissions? En caso de necesitarlo utiliza la definida por el framework, voy a dejarte un enlace para que veas que tu definición es la misma que está en el framework . https://github.com/encode/django-rest-framework/blob/master/rest_framework/permissions.py

Edited 2023-06-15 14:51:22 UTC

Martin se lo hizo porque class DjangoModelPermissions(BasePermission), no estaba especificado el metodo get

# Map methods into required permission codes.
# Override this if you need to also provide 'view' permissions,
# or if you want to provide custom permission codes.
perms_map = {
    'GET': [],
    'OPTIONS': [],
    'HEAD': [],
    'POST': ['%(app_label)s.add_%(model_name)s'],
    'PUT': ['%(app_label)s.change_%(model_name)s'],
    'PATCH': ['%(app_label)s.change_%(model_name)s'],
    'DELETE': ['%(app_label)s.delete_%(model_name)s'],
}

Por favor elimina la importación de CustomModelPermissions esto si no lo estamos utilizando.

Edited 2023-06-15 14:50:23 UTC

Por favor elimina la propiedad permission_classes esto si no se está utilizando.

Edited 2023-06-15 14:49:46 UTC

Por favor elimina este archivo que no se utiliza, los tests los estás definiendo en el directorio tests de la aplicación organismo.

Added 1 commit:

• 55952da0 - Modificado las modificaciones solicitadas

Status changed to merged

mentioned in commit 1d0f41cb