-
Notifications
You must be signed in to change notification settings - Fork 26
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
Vendor name e alcune traduzioni #16
base: master
Are you sure you want to change the base?
Conversation
Salve @arioch1984 , Mettiamo la pull request a revisione nel gruppo e poi proseguiamo con l'approvazione. |
Grazie a te Giuseppe :)
Non ho mai ricevuto una risposta così gentile ad una pull request e grazie per aver preso in considerazione la mia richiesta.
In ultimo abbiate pietà perchè vedrete un paio di errori di sintassi in due commenti allegati a dei commit, mi ero sbagliato ma non me ne ero accorto :)
Fabio Brunelli
http://www.fabiobrunelli.it
…On 15 May 2018, 00:03 +0200, Giuseppe Morelli ***@***.***>, wrote:
Salve @arioch1984 ,
grazie per la tua pull request e per il tuo contributo.
Il discorso del vendor it4mage e itformage è dovuto dal fatto che nel registration.php non viene accettato un modulo con il numero al suo interno (o per lo meno era legato a quello)
Di questo se ne era occupato @WaPoNe, credo quindi che per la parte del composer.json verrà fatto un rollback (è stato concordato di utilizzare itformage).
Mettiamo la pull request a revisione nel gruppo e poi proseguiamo con l'approvazione.
Grazie
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Salve...
...Nel gruppo dove lavoro ci siamo permessi di modificare alcune traduzioni e soprattutto allineare il vendor name del modulo che installato con composer andava nella cartella "it4mage" sotto vendor ma aveva "itformage" nel registrato.php e itfm nel language.xml
Sperando di fare cosa gradita propongo le modifiche da noi fatte come full request.