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

[FIX] Non deve essere possibile creare o modificare i tipi di documento fiscale #3535

Conversation

SirTakobi
Copy link
Contributor

@SirTakobi SirTakobi commented Aug 24, 2023

Corregge #3532 per 12.0.
Sostituisce #2819.

@SirTakobi SirTakobi marked this pull request as ready for review August 24, 2023 09:05
Copy link
Member

@eLBati eLBati left a comment

Choose a reason for hiding this comment

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

Grazie

'depends': ['l10n_it_account'],
'depends': [
'l10n_it_account',
'mail',
Copy link
Member

Choose a reason for hiding this comment

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

come mai vuoi aggiungere questa dipendenza?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Perché nel test uso mail_new_test_user che pensavo venisse dai test del modulo mail, ora però vedo che viene dal modulo test_mail che in effetti contiene proprio i test per il modulo mail; quindi mettere uno o l'altro potrebbe essere equivalente.
Se preferisci metto test_mail invece di mail.

Copy link
Member

Choose a reason for hiding this comment

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

Se preferisci metto test_mail invece di mail.

sì preferirei grazie

Copy link
Contributor

@SirAionTech SirAionTech Oct 5, 2023

Choose a reason for hiding this comment

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

Se preferisci metto test_mail invece di mail.

sì preferirei grazie

Fatto in #3535 #3640

@eLBati
Copy link
Member

eLBati commented Oct 5, 2023

#3640

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.

5 participants