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

Aggiunta ritenuta d'acconto predefinita per anagrafica #305

Merged
merged 5 commits into from
Sep 26, 2018
Merged

Aggiunta ritenuta d'acconto predefinita per anagrafica #305

merged 5 commits into from
Sep 26, 2018

Conversation

maicol07
Copy link
Contributor

@maicol07 maicol07 commented Sep 23, 2018

Descrizione

  • Aggiunta ritenuta d'acconto predefinita per anagrafica (cliente/fornitore). Il funzionamento è identico a quello dell'IVA predefinita.

Risolve: #155 (comment), #275

Tipologia

  • Nuova funzionalità (cambiamenti minori che aggiungono una nuova funzionalità)

Checklist

  • Il codice segue le linee guida del progetto
  • Ho commentato il codice, in particolare nelle parti più complesse
  • Ho aggiornato di conseguenza la documentazione (se presente) [NON NECESSARIO]
  • Il codice non genera warnings

@maicol07 maicol07 changed the title Aggiunta ritenuta d'acconto predefinita per anagrafica (cliente/fornitore) Aggiunta ritenuta d'acconto predefinita per anagrafica Sep 23, 2018
@maicol07
Copy link
Contributor Author

@Dasc3er cosa ne pensi della PR? Può essere aggiunta come funzionalità per la 2.4.2?

@Dasc3er
Copy link
Contributor

Dasc3er commented Sep 25, 2018

@maicol07 Sembra una buona proposta.
Potresti migliorare la struttura degli identificatori utilizzati?
Ogni parola dovrebbe essere separata da degli underscore 😄

@Dasc3er Dasc3er added the miglioria Proposte di miglioramenti label Sep 25, 2018
@Dasc3er Dasc3er added this to the OSM 2.4.2 milestone Sep 25, 2018
@maicol07
Copy link
Contributor Author

Certamente, per quanto riguarda i conflitti? Devo ancora vedere bene le modifiche effettuate nel file actions.php

@Dasc3er
Copy link
Contributor

Dasc3er commented Sep 25, 2018

Li ho risolti, ho fatto in giornata un aggiornamento sulla gestione delle anagrafiche per iniziare il passaggio ad Eloquent (#102).

@maicol07
Copy link
Contributor Author

@Dasc3er non ho capito cosa dovrei cambiare, tutti gli altri nomi di id (es. idiva) non hanno gli _, tranne per la distinzione tra acquisti e vendite...

@Dasc3er
Copy link
Contributor

Dasc3er commented Sep 25, 2018

Dovresti aggiungere gli underscore nei nuovi campi e variabili da te inseriti, poiché lentamente stiamo adeguando il codice per renderlo più leggibile e standardizzato.

@maicol07
Copy link
Contributor Author

@Dasc3er fatto

@Dasc3er Dasc3er merged commit 40d1b95 into devcode-it:master Sep 26, 2018
@Dasc3er Dasc3er mentioned this pull request Sep 26, 2018
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
miglioria Proposte di miglioramenti
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants