Skip to content
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

Formulario para la gestión de las secciones de la tabla de contenidos de los NÚMEROS de las revistas #727

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

eduranm
Copy link
Collaborator

@eduranm eduranm commented Apr 7, 2024

O que esse PR faz?

Creación de nuevo formulario para la gestión de las secciones de la tabla de contenidos de los NÚMEROS de las revistas.

Onde a revisão poderia começar?

Por commit

Como este poderia ser testado manualmente?

Ingreso a formulario IssueTOC

Algum cenário de contexto que queira dar?

N/A

Screenshots

image

Quais são tickets relevantes?

#628

Referências

N/A

toc/models.py Outdated
related_name="journal_section",
)

section = models.TextField(_("Section"), null=True, blank=True)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eduranm creo que es importante indica la lengua de la sección. Pienso que una forma de solucionar el problema que le había yo comentado es añadir um ForeignKey / Autocomplete para self (la propia clase)

Comment on lines +87 to +91
"JournalSectionCode",
verbose_name=_("Journal Section Code"),
blank=True,
)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eduranm añadir también para JournalSection

class IssueTOCAdmin(ModelAdmin):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Utilizar wagtail snippets

@samuelveigarangel
Copy link
Collaborator

@eduranm É interessante resumir essas migrações em apenas uma.

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.

3 participants