Skip to content

[8.0][IMP] Opzione per stampare il codice fiscale#575

Closed
alessandrocamilli wants to merge 1 commit into
OCA:8.0from
alessandrocamilli:8.0-l10n_it_vat_registries_fiscalcode
Closed

[8.0][IMP] Opzione per stampare il codice fiscale#575
alessandrocamilli wants to merge 1 commit into
OCA:8.0from
alessandrocamilli:8.0-l10n_it_vat_registries_fiscalcode

Conversation

@alessandrocamilli

Copy link
Copy Markdown
Contributor

Sul wizard di stampa viene aggiunta l'opzione che permette di stampare nelle info riguardanti il partner anche il codice fiscale

@eLBati eLBati left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. @alessandrocamilli anche qua, puoi fare anche per v10?

@eLBati eLBati added this to the 8.0 milestone Aug 28, 2018
@eLBati eLBati added needs review and removed 8.0 labels Aug 28, 2018
@SimoRubi

Copy link
Copy Markdown
Member

@alessandrocamilli puoi creare una issue per tracciare lo stato della modifica anche nelle altre versioni?
Il template è https://github.com/OCA/l10n-italy/wiki/Team-di-sviluppo

@SimoRubi SimoRubi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Solo un appunto, ma per il resto è ok (code review)

<th class="left_without_line">RAGIONE SOCIALE</th>
<th class="left_without_line">P.IVA</th>
<th class="left_without_line">P.IVA
<t t-if="fiscalcode == True">

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Qui e sotto, dovrebbe essere sufficiente:

Suggested change
<t t-if="fiscalcode == True">
<t t-if="fiscalcode">

@alessandrocamilli

alessandrocamilli commented Aug 16, 2019 via email

Copy link
Copy Markdown
Contributor Author

@SimoRubi

Copy link
Copy Markdown
Member

fatto! Il giorno ven 16 ago 2019 alle ore 13:20 Simone Rubino < notifications@github.com> ha scritto:

@SimoRubi approved this pull request. Solo un appunto, ma per il resto è ok (code review)

@alessandrocamilli non ci sono cambiamenti rispetto a quando ti ho scritto

@github-actions

Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Oct 24, 2021
@github-actions github-actions Bot closed this Nov 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved needs review stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants