Skip to content

Conversation

@eduranm
Copy link
Contributor

@eduranm eduranm commented Feb 5, 2025

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
image

Quais são tickets relevantes?
#12

Referências
N/A

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(
Copy link
Member

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 (
Copy link
Member

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

Copy link
Member

@robertatakenaka robertatakenaka left a 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

@robertatakenaka robertatakenaka merged commit 938b81e into scieloorg:main Feb 5, 2025
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants