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

16.0 mig l10n_it_delivery_note_batch #3032

Closed
wants to merge 58 commits into from

Conversation

jado95
Copy link
Contributor

@jado95 jado95 commented Nov 11, 2022

Dipende da #3020

Borruso and others added 30 commits November 11, 2022 14:48
Currently translated at 100.0% (9 of 9 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note_batch
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note_batch/it/
Currently translated at 95.1% (59 of 62 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_delivery_note_base
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_delivery_note_base/it/
Currently translated at 63.4% (40 of 63 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note_base
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note_base/it/
Currently translated at 100.0% (63 of 63 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note_base
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note_base/it/
Currently translated at 100.0% (63 of 63 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note_base
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note_base/it/
Currently translated at 89.2% (222 of 249 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_delivery_note
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_delivery_note/it/
Currently translated at 98.7% (246 of 249 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-l10n_it_delivery_note
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-l10n_it_delivery_note/it/
…invoiced' where there are no sale order lines

According to https://docs.python.org/3/library/functions.html#all:

all(iterable)

Return True if all elements of the iterable are true (or if the iterable is empty).
Currently translated at 79.8% (206 of 258 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-l10n_it_delivery_note
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-l10n_it_delivery_note/it/
This fixes the following issue:

Steps to reproduce:

* create a SO with two lines and confirm it
* delivery partially only one of the two lines
* validate the backorder

Current behavior:

The backorder is not validated as stock_delivery_note_line_move_uniq constraint is raised

Expected behavior:

The backorder is validated
@jado95 jado95 force-pushed the 16.0-mig-l10n_it_delivery_note_batch branch from 632534e to 1f5d6a3 Compare November 11, 2022 13:54
@jado95 jado95 mentioned this pull request Nov 11, 2022
79 tasks
@eLBati
Copy link
Member

eLBati commented Nov 11, 2022

/ocabot migration l10n_it_delivery_note_batch

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Nov 11, 2022
Copy link
Contributor

@TonyMasciI TonyMasciI left a comment

Choose a reason for hiding this comment

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

Eseguita solo revisione tecnica

  • Perché sono presenti i moduli l10n_it_delivery_note_ & l10n_it_delivery_note_base in questa PR?
  • Installazione nessun problema

@jado95 jado95 force-pushed the 16.0-mig-l10n_it_delivery_note_batch branch from d6a2fdc to 64e1fc6 Compare February 17, 2023 09:37
@jado95
Copy link
Contributor Author

jado95 commented Feb 17, 2023

Eseguita solo revisione tecnica

* Perché sono presenti i moduli l10n_it_delivery_note_ & l10n_it_delivery_note_base in questa PR?

* Installazione nessun problema

C'erano per errore, adesso dovrebbe essere ok.

@jado95 jado95 force-pushed the 16.0-mig-l10n_it_delivery_note_batch branch from 64e1fc6 to 1d49f7b Compare March 3, 2023 08:40
Copy link
Contributor

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

LGTM

@tafaRU
Copy link
Member

tafaRU commented Jul 27, 2023

@jado95 puoi risolvere i conflitti?

@tafaRU
Copy link
Member

tafaRU commented Sep 7, 2023

@jado95 #3032 (comment)

@jado95 jado95 force-pushed the 16.0-mig-l10n_it_delivery_note_batch branch from 1d49f7b to 7a0f9eb Compare September 7, 2023 15:49
@tafaRU
Copy link
Member

tafaRU commented Sep 7, 2023

@jado95 puoi schiacciare i commit dei bot con i precedenti?

@tafaRU
Copy link
Member

tafaRU commented Sep 8, 2023

@jado95 puoi schiacciare i commit dei bot con i precedenti?

@jado95 come da deciso in call stamattina provvedo io a schiacciarli.

@tafaRU
Copy link
Member

tafaRU commented Sep 8, 2023

Ho notato che ci sono delle modifiche che impattano anche altri moduli:

image

Temo che sia meglio rifarla from scratch.

[EDIT]

Anche nel commit di migrazione (7a0f9eb) ci sono modifiche che coinvolgono l10n_it_delivery_note.

@OpenCode
Copy link
Contributor

OpenCode commented Oct 5, 2023

Sostituita da #3636

@OpenCode OpenCode closed this Oct 5, 2023
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.