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.

Sign up for GitHub

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

Jump to bottom

[14.0][MIG] l10n it asset history import #3227

Closed
wants to merge 7 commits into from
Closed

[14.0][MIG] l10n it asset history import #3227

wants to merge 7 commits into from

Conversation

odooNextev
Copy link
Contributor

No description provided.

@odooNextev odooNextev mentioned this pull request Mar 23, 2023
Closed
@matteoopenf
Copy link
Contributor

Due appunti, oltre che manca la history il pot non va riesportato ci pensa Ocabot e farlo al merge della PR

@odooNextev odooNextev changed the title 14.0 mig l10n it asset history import [14.0][MIG] l10n it asset history import Mar 23, 2023
@odooNextev
Copy link
Contributor Author

Due appunti, oltre che manca la history il pot non va riesportato ci pensa Ocabot e farlo al merge della PR

Per il pot va bene, mentre la history c'è perchè Matteo ha aperto la PR con un nuovo branch creato 25 minuti fa e nel frattempo sono stati fatti solo quei commit

@matteoopenf
Copy link
Contributor

Due appunti, oltre che manca la history il pot non va riesportato ci pensa Ocabot e farlo al merge della PR

Per il pot va bene, mentre la history c'è perchè Matteo ha aperto la PR con un nuovo branch creato 25 minuti fa e nel frattempo sono stati fatti solo quei commit

non c'è ecco perchè te l'ho scritto, perchè il primo commit dovrebbe essere il mio appunto perchè mantieni la storia.
Ti prendo una pr di esempio, perchè vero che si chiama Matteo come me, ma non viene mantenuta la paternità del mio commit:
Guarda per esempio questa migration #3214 e solo l'ultimo commit è di chi la fa con i fix che occorrono i primi sono di chi ha lavorato al modulo prima di lui

@odooNextev odooNextev closed this Mar 23, 2023
@primes2h
Copy link
Contributor

il pot non va riesportato ci pensa Ocabot e farlo al merge della PR

Vedi #3204 (comment)
Aggiungo che avere il .pot aggiornato in una PR di migrazione aiuta molto nella revisione delle stringhe sorgente, dal momento che sono tutte raccolte in un unico file.

@odooNextev odooNextev deleted the 14.0-mig-l10n_it_asset_history_import branch June 1, 2023 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@odooNextev @matteoopenf @primes2h @OCA-git-bot @francesco-ooops @matteonext