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

[12.0][MIG] l10n_it_ricevute_bancarie #1564

Merged
merged 56 commits into from
Jan 20, 2020
Merged

[12.0][MIG] l10n_it_ricevute_bancarie #1564

merged 56 commits into from
Jan 20, 2020

Conversation

tafaRU
Copy link
Member

@tafaRU tafaRU commented Dec 20, 2019

Sono partito dalla #1554 preservando la history.

@tafaRU tafaRU mentioned this pull request Dec 20, 2019
Closed
stevech091
stevech091 approved these changes Dec 21, 2019
View reviewed changes
Copy link

@stevech091 stevech091 left a comment

Choose a reason for hiding this comment

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

test functional ok

@MarcoCalcagni
Copy link
Contributor

Sono partito dalla #1554 preservando la history.

Grazie per vari impegni non sono riuscito a farlo.

@stevech091
Copy link

@As400it riesci a fargli la review funzionale?
Grazie

@primes2h
Copy link
Contributor

Ho fatto una revisione generale della PR, le modifiche che propongo sono qui tafaRU#8.

@primes2h
Copy link
Contributor

primes2h commented Jan 2, 2020

🤔 dov'è finito il runbot?

eLBati and others added 21 commits January 3, 2020 09:15
@eLBati @tafaRU
[ADD] l10n_it_ricevute_bancarie
918370f 
moving from https://code.launchpad.net/~openobject-italia-core-devs/openobject-italia/l10n_it_ricevute_bancarie
@eLBati @tafaRU
[IMP] l10n_it_ricevute_bancarie splitted in l10n_it_ricevute_bancarie…
3bd7ebc 
…_jasper
@scigghia @tafaRU
[FIX] classe riba_distinta_move_line e context_today sconosciuto in 6\.0
d16e95d
@eLBati @tafaRU
[FIX] Reverted to previous revision.
9497226 
Changes for 6.0 should be done in a branch for 6.0.
For the reason to use 'fields.date' instead of 'time', see
https://bugs.launchpad.net/openobject-server/+bug/925361
https://bugs.launchpad.net/openerp-web/+bug/918257
@eLBati @tafaRU
[FIX] removing useless rows
1ee5311
@eLBati @tafaRU
[IMP] copyright
5827258
@eLBati @tafaRU
[REM] l10n_it_ricevute_bancarie*
5ae7438
@dcorio @tafaRU
[ADD] riba v7
dc860be
@nicothebrush @tafaRU
Inserito fix richiesto da eLBati
97d1c32
@eLBati @tafaRU
[FIX] https://code.launchpad.net/~openobject-italia-core-devs/openobj…
920fdcc 
…ect-italia/adding_riba_v7/+merge/194620/comments/449412
@eLBati @tafaRU
[FIX] AttributeError: 'cursor' object has no attribute 'pool'
a78bd99 
and
valueerror
@eLBati @tafaRU
[IMP] PEP8
b963125
@eLBati @tafaRU
[FIX] osv.osv
b618e08
@eLBati @tafaRU
[IMP] PEP8
1d75a89
@eLBati @tafaRU
[IMP] PEP8
f1b801a
@eLBati @tafaRU
[REM] reverting PEP8 improvements.
9018390 
Moving to lp:~agilebg/openobject-italia/riba_v7_style
@scigghia @tafaRU
[imp] l10n_it_ricevute_bancarie\: controllo presenza banca cliente in…
1e8f984 
… wizard export riba
@scigghia @tafaRU
[fix] l10n_it_ricevute_bancarie: lunghezza città banca max 23 caratte…
12e25e1 
…ri in wizard export
@scigghia @tafaRU
[imp][l10n_it_ricevute_bancarie] controllo presenza su conto accettaz…
d376240 
…ione, data di registrazione su distinta
@eLBati @tafaRU
[REF] Set all the modules of the master branches to "installable": False
fd876dc 
moves all modules into a __unported__ directory
@sbidoul @tafaRU
[MOV] move addons out of __unported__ (they remain not installable)
bcfc324
@tafaRU
Copy link
Member Author

tafaRU commented Jan 3, 2020 •
edited
Loading

ho dovuto fare rebase per via via di un conflitto sul file oca_dependencies.txt.

@tafaRU
Copy link
Member Author

tafaRU commented Jan 3, 2020

dov'è finito il runbot?

@primes2h vedi OCA/web#1467 (comment)

@primes2h
Copy link
Contributor

primes2h commented Jan 4, 2020

🤔 dov'è finito il runbot?

dov'è finito il runbot?

@primes2h vedi OCA/web#1467 (comment)

@tafaRU grazie per l'info!

@primes2h
Copy link
Contributor

primes2h commented Jan 4, 2020

Peek 04-01-2020 20-19

@primes2h
Copy link
Contributor

primes2h commented Jan 4, 2020

Peek 04-01-2020 20-22

@stevech091
Copy link

@primes2h non ho capito il tuo video a che cosa si riferisce.
Grazie

@primes2h
Copy link
Contributor

primes2h commented Jan 7, 2020

@primes2h non ho capito il tuo video a che cosa si riferisce.
Grazie

Quale dei due non ti è chiaro?

@stevech091
Copy link

Sinceramente tutti e 2 e non riesco ad inquadrarli nelle procedure RIBA.
Grazie

@primes2h
Copy link
Contributor

primes2h commented Jan 7, 2020 •
edited
Loading

Sinceramente tutti e 2 e non riesco ad inquadrarli nelle procedure RIBA.
Grazie

Ah ok.
Allora, nel primo video volevo mettere in evidenza che in fase di inserimento una volta selezionato il "Journal Entry" non è più possibile modificarlo, di conseguenza viene anche inibita la selezione del conto.
È possibile solo annullare la creazione dell'emissione RiBa.

Nel secondo mettevo in evidenza la comparsa di un errore Odoo, anche se non è la procedura corretta quel tipo di errore dovrebbe essere gestito con una finestra di avviso.

@stevech091
Copy link

ok ma non capisco nel flusso delle riba dove si andrebbero "collocate" le tue operazioni?
Grazie

@primes2h
Copy link
Contributor

primes2h commented Jan 7, 2020 •
edited
Loading

ok ma non capisco nel flusso delle riba dove si andrebbero "collocate" le tue operazioni?
Grazie

Non ho seguito il flusso Ri.Ba. indicato nel README.
Volevo solo mettere in evidenza il fatto che da un punto di vista funzionale se un utente fa un errore non compilando correttamente qualche campo:

  1. dovrebbe avere la possibilità di modificarlo al volo dopo averlo inserito e prima di averlo salvato (primo video riguardante l'emissione RiBa) [*]
  2. non dovrebbero esserci crash di Odoo ma messaggi di avviso che indichino cosa manca o cosa non è stato compilato correttamente. (secondo video)

[*] Nella creazione delle distinte invece è possibile.

@marco-marchiori
Copy link

@primes2h c'e' sicuramente un problema in quella funzionalita', ma a me risulta si porti dietro da molte versioni e, in quel caso specifico e' proprio un problema di fondo, che forse consiglierebbe di inibire la creazione, in queste condizioni. Sarebbe senz'altro auspicabile un refactoring di tutto il modulo ma mi pare che l'obiettivo comune fosse limitato a consentire l'utilizzo delle riba ai clienti con scritture corrette utilizzando la proc. standard.
Per quanto mi riguarda ho testato in locale il modulo funziona, quindi test funzionale OK.

@tafaRU
Copy link
Member Author

tafaRU commented Jan 17, 2020 •
edited
Loading

@primes2h per prevenire i problemi da te segnalati sopra, nella chiamata di OCA/l10-italy, si è deciso di applicare 959039f.
Puoi verificare se per te è ok?

@primes2h
Copy link
Contributor

@primes2h per prevenire i problemi da te segnalati sopra, nella chiamata di OCA/l10-italy, si è deciso di applicare 959039f.
Puoi verificare se per te è ok?

👌
È la soluzione più semplice e immediata direi, la filosofia KISS è sempre una garanzia. 😄
Grazie!

@primes2h
Copy link
Contributor

@primes2h c'e' sicuramente un problema in quella funzionalita', ma a me risulta si porti dietro da molte versioni e, in quel caso specifico e' proprio un problema di fondo, che forse consiglierebbe di inibire la creazione, in queste condizioni.

@marco-marchiori
Inibire la creazione come fatto in 959039f mi sembra un'ottima soluzione al momento. 👍

marco-marchiori
marco-marchiori approved these changes Jan 20, 2020
View reviewed changes
Copy link

@marco-marchiori marco-marchiori left a comment

Choose a reason for hiding this comment

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

Ho testato l'emissione diretta ed in effetti e' inibita. Rimane quindi possibile emettere ricevute solo con il processo che parte dalla fattura correttamente configurata.

@eLBati
Copy link
Member

eLBati commented Jan 20, 2020

/ocabot merge

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-1564-by-eLBati-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 20, 2020
@OCA-git-bot
Merge PR #1564 into 12.0
7efd38c 
Signed-off-by eLBati
@OCA-git-bot OCA-git-bot merged commit 959039f into OCA:12.0 Jan 20, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at d68ae47. Thanks a lot for contributing to OCA. ❤️

@tafaRU tafaRU deleted the 12.0-mig-l10n_it_ricevute_bancarie branch January 21, 2020 08:04
@OCA-git-bot OCA-git-bot mentioned this pull request Jan 30, 2020
Closed
35 tasks
@primes2h primes2h mentioned this pull request Mar 29, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marco-marchiori marco-marchiori marco-marchiori approved these changes

@MarcoCalcagni MarcoCalcagni MarcoCalcagni approved these changes

@stevech091 stevech091 stevech091 approved these changes

@primes2h primes2h primes2h approved these changes

Assignees
No one assigned
Labels
approved merged 🎉 migration needs review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.