-
Notifications
You must be signed in to change notification settings - Fork 5
Área administrativa para reference #14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| for linea in lineas: | ||
| linea = linea.strip() # Eliminar espacios adicionales en cada línea | ||
| if linea: # Evitar procesar líneas vacías | ||
| new_reference = Reference.objects.create( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eduranm Es necesario identificar si ya existe antes de crear. Puedes usar IntegrityError que es una excepción que ocurre al ingresar un contenido existente.
| @@ -0,0 +1,55 @@ | |||
| from django.http import HttpResponseRedirect | |||
| from django.utils.translation import gettext as _ | |||
| from wagtail_modeladmin.options import ( | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eduranm verifica el proyecto usage que ya no usa más modeladmin:
https://github.com/scieloorg/usage/blob/main/log_manager/wagtail_hooks.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@eduranm estoy aprobando aunque haya algunos cambios solicitados
O que esse PR faz?
Agrega área administrativa para el modelo reference
Onde a revisão poderia começar?
Por commit
Como este poderia ser testado manualmente?
Ingresa al menú reference
Algum cenário de contexto que queira dar?
N/A
Screenshots

Quais são tickets relevantes?
#12
Referências
N/A