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

[UPD] Upgrade POS modules to 14.0 #2796

Closed
wants to merge 1 commit into from
Closed

[UPD] Upgrade POS modules to 14.0 #2796

wants to merge 1 commit into from

Conversation

AIR-Leo
Copy link

@AIR-Leo AIR-Leo commented May 13, 2022

Ciao!

Ho provato a riassumere in questa PR la migrazione fatta dei moduli POS dal nostro team negli ultimi mesi.

Comprende:

  • fiscal_epos_print: Driver per la connessione con stampanti Epson, con la gestione di resi, sconti, ticket;

  • fiscal_epos_print_fiscalcode: Per la stampa del codice fiscale sugli scontrini, secondo le indicazioni ufficiali di Epson e gli aggiornamenti della normativa (viene trasmesso via , per la trasmissione all'Agenzia);

  • l10n_it_pos_fatturapa: Inserimento del codice SDI all'interno dell'interfaccia POS. Realizzato da SteSi;

  • l10n_it_pos_fiscalcode: Inserimento del codice fiscale all'interno dell'interfaccia POS;

  • pos_partner_firstname: Gestione del nome/cognome all'interno dell'interfaccia POS.

È la mia prima PR: spero di aver seguito correttamente i passaggi, nel caso sono disponibile per qualsiasi correzione :)

Buon lavoro!

Leo

@OCA-git-bot
Copy link
Contributor

Hi @eLBati,
some modules you are maintaining are being modified, check this out!

@matteoopenf
Copy link
Contributor

l'ultimo modulo credo sia un duplicato OCA/pos#775 ci sono inoltre già altre pr che migrano questi moduli se fai review li andiamo avanti?


@api.constrains('fpdeptax')
def _validate_fpdeptax(self):
if not re.search(regex, self.fpdeptax):
Copy link
Contributor

Choose a reason for hiding this comment

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

in questo punto puoi aggiungere un for tax in self: perché mi sono accorto che capita che va in errore per expected singleton


@api.model
def create_from_ui(self, orders, draft=False):
res = super(PosOrder, self).create_from_ui(orders)
Copy link
Contributor

Choose a reason for hiding this comment

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

res = super(PosOrder, self).create_from_ui(orders, draft=draft) il super dovrebbe essere così

@micheledic
Copy link
Contributor

Buongiorno,
mancano due lib nel commit nel modulo fiscal_epos_print,
risultano esserci gli import nel file assets ma mancano i file

<script type="text/javascript" src="/fiscal_epos_print/static/lib/fiscalprint/fiscalprint.js"/> <script type="text/javascript" src="/fiscal_epos_print/static/lib/pikaday/pikaday.min.js"/> Probabilmente per la presenza di **lib/** nel .gitignore

@AIR-Leo
Copy link
Author

AIR-Leo commented Jun 9, 2022

Ciao a tutti,

Scusate per la domanda magari banale. In questi casi conviene fare una nuova PR?

@matteoopenf
Copy link
Contributor

Ciao a tutti,

Scusate per la domanda magari banale. In questi casi conviene fare una nuova PR?

No aggiorni questa, altrimenti diventano una moltitudine di pr e si perde il filo

@matteoopenf
Copy link
Contributor

per caso ti funziona anche la stampa da IPAD/iphone?

@AIR-Leo
Copy link
Author

AIR-Leo commented Jul 18, 2022

Ciao @matteoopenf, funziona solo se la stampante Epson ha un certificato SSL valido. Se non c'è il certificato, c'è da abilitare su Chrome/Firefox l'autorizzazione a vedere i "contenuti non sicuri", cosa che (attualmente) credo non si possa fare su ios.

@matteoopenf
Copy link
Contributor

Grazie innanzitutto per la risposta.
Ho comprato un certificato da namecheap e l'ho caricato da upload.cgi ma ancora mi deve mancare qualcosa

@matteoopenf
Copy link
Contributor

Hai per caso una guida da seguire per il certificato ecc che così cerco di capire cosa mi è sfuggito?

@AIR-Leo
Copy link
Author

AIR-Leo commented Jul 18, 2022

Ciao @matteoopenf, per velocità ci sentiamo via Discord? Mi trovi come LeonardoGuerra#8849

@masimassimo
Copy link

Hai per caso una guida da seguire per il certificato ecc che così cerco di capire cosa mi è sfuggito?

Ciao @matteoopenf , se vuoi ci possiamo sentire su discord nel canale Odoo Italia

@matteoopenf
Copy link
Contributor

Hai per caso una guida da seguire per il certificato ecc che così cerco di capire cosa mi è sfuggito?

Ciao @matteoopenf , se vuoi ci possiamo sentire su discord nel canale Odoo Italia

Buondi, mi diresti il tuo nick?

@micheledic
Copy link
Contributor

salve, ci sono update in merito a questa PR o qualcuno sta aggiornando fiscal_epos_print ?

@francesco-ooops
Copy link
Contributor

@OCA/local-italy-maintainers penso pure questa si possa chiudere

@tafaRU tafaRU closed this Jul 14, 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.

8 participants