-
Notifications
You must be signed in to change notification settings - Fork 0
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
added Documento ct #5
Conversation
), | ||
) | ||
|
||
# formati_alternativi = BlocksField( |
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.
Direi che qui ci vanno 3 campi file come questi: https://github.com/RedTurtle/design.plone.contenttypes/blob/main/src/design/plone/contenttypes/behaviors/multi_file.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.
Fatto.
) | ||
|
||
uo_correlata = RelationList( | ||
title=_("uo_correlata_documento_label", default="Responsabile del documento"), |
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.
Metti una descrizione per dire che si fa riferimento ad una UO
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.
Fatto.
@@ -24,6 +24,11 @@ class FieldsetsMismatchError(Exception): | |||
"utenti", | |||
"ulteriori_informazioni", | |||
], | |||
"Documento": [ |
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.
Non so se i file (principale e alternativi) li metterei in un fieldest a parte, subito dopo default
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.
Li ho messi nel fieldset "Formati".
"title", | ||
"descrizione_estesa", | ||
"uo_correlata", | ||
# "description", is required from schema_tweaks.py but it doesn't apply in test |
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.
non è obbligatorio anche il file principale? Il campo "vai al documento" secondo me è il file principale, chiamato "file"
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.
Ah, il campo file, marcalo come primario (guarda come è fatto per il ct File)
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.
Fatto.
], | ||
) | ||
|
||
def test_documento_fields_a_chi_si_rivolge_fieldset(self): |
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.
Ti manca il test al secondo fielsdet
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.
Fatto.
Aggiunto il ct Documento.
I campi "Vai al documento" e "Formati alternativi" devono essere due campi file o si intende che alla creazione del documento vengono generate le subfolder per contenere quei file?