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

[12.0] ADD fiscal_epos_print #1557

Merged
merged 8 commits into from
Jan 18, 2020
Merged

Conversation

eLBati
Copy link
Member

@eLBati eLBati commented Dec 10, 2019

v10: #1558

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@eLBati eLBati force-pushed the 12.0-fiscal_epos_print branch 2 times, most recently from 76b515b to 9ab53ee Compare December 12, 2019 08:23
Copy link
Contributor

@robyf70 robyf70 left a comment

Choose a reason for hiding this comment

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

La validazione la sposterei da un'altra parte

@eLBati eLBati force-pushed the 12.0-fiscal_epos_print branch 3 times, most recently from e3a8b10 to e4578a2 Compare December 14, 2019 05:56
@eLBati eLBati force-pushed the 12.0-fiscal_epos_print branch 8 times, most recently from ce9ac3b to a9c5656 Compare December 17, 2019 16:38
@GSLabIt
Copy link
Contributor

GSLabIt commented Dec 21, 2019

Non ho ben compreso il compute_all, le tasse non vengono già calcolate?

Non ho trovato la stampa del z report

@eLBati
Copy link
Member Author

eLBati commented Dec 23, 2019

Non ho ben compreso il compute_all, le tasse non vengono già calcolate?

Vedi

// Intervene here: also add taxes with 0 amount

dobbiamo intervenire per aggiungere a list_taxes anche le imposte con aliquote 0, altrimenti ignorate da odoo core.

Non ho trovato la stampa del z report

In effetti manca.
Provo ad aggiungere

@eLBati eLBati force-pushed the 12.0-fiscal_epos_print branch 2 times, most recently from 5ec48a7 to c21eae6 Compare January 14, 2020 14:31
Copy link
Contributor

@primes2h primes2h left a comment

Choose a reason for hiding this comment

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

Revisione documentale veloce.

P.s. nel README ci sono alcune parti in inglese (CONFIGURE e DESCRIPTION) e alcune in italiano (ROADMAP)

fiscal_epos_print/__manifest__.py Show resolved Hide resolved
fiscal_epos_print/static/src/xml/pos.xml Outdated Show resolved Hide resolved
fiscal_epos_print/static/src/js/epson_epos_print.js Outdated Show resolved Hide resolved
fiscal_epos_print/models/account_journal.py Outdated Show resolved Hide resolved
fiscal_epos_print/static/src/xml/pos.xml Outdated Show resolved Hide resolved
fiscal_epos_print/static/src/xml/pos.xml Outdated Show resolved Hide resolved
fiscal_epos_print/static/src/js/chrome.js Outdated Show resolved Hide resolved
@eLBati
Copy link
Member Author

eLBati commented Jan 15, 2020

@primes2h grazie, fatto (ROADMAP essendo temporaneo potremmo anche lasciarlo in italiano, cosa dici?)

@primes2h
Copy link
Contributor

@primes2h grazie, fatto (ROADMAP essendo temporaneo potremmo anche lasciarlo in italiano, cosa dici?)

👍

Copy link
Contributor

@primes2h primes2h left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@GSLabIt GSLabIt left a comment

Choose a reason for hiding this comment

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

GO FOR IT

Copy link
Contributor

@robyf70 robyf70 left a comment

Choose a reason for hiding this comment

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

@eLBati il termine Report reference è più giusto che sia Riferimento chiusura per il resto LGTM

eLBati and others added 8 commits January 18, 2020 07:50
[10.0][fiscal_epos_print] Renamed Cassa to RT in the refund dialog

[10.0][fiscal_epos_print] Check if the update_refund_info_button property has been passed before to call it

[10.0][fiscal_epos_print] Add support of pos.config header and footer when set

[fiscal_epos_print] Export refund_info_button widget for inheritance purposes

Add support for payment type index

Allow to configure whether to Show receipt preview or not
trans

FIX journal form showing fiscalprinter data for bank journals

Allow to use tax exclude prices in products and force to set "Tax-Included Price" in pos config

REF file structure splitting fp90iii.js

Move "Wrong tax configuration" check before creating receipt

Update library ePOS-Print and Fiscal Print API

IMP fiscal_epos_print writing fiscal receipt data to pos.order

show_popup instead of alert
…ceipt number

Lock receipt screen until printer communication ends.
Prevent user to go to next order if receipt not correctly printed.
Allow user to retry printing in case of errors

ADD print fiscal closure widget (Z report)
and X report

ADD refund management from OrderListScreenWidget
FIX Restaurant compatibility
[10.0][fiscal_epos_print] Add reprint last receipt

[10.0][fiscal_epos_print] Show subtotal in the printer display when showing the payment screen

[10.0][fiscal_epos_print] Reorganize the onreceive() to simplify it

[10.0][fiscal_epos_print] Add cashdrawer automatic opening from pos config and from widget

[10.0][fiscal_epos_print] Rework ePOS widgets into a more versatile floating window
Fix fiscal_receipt_date format before to sent to backend
To avoid risk to write date in wrong format before 'update_fiscal_receipt_values'
@eLBati
Copy link
Member Author

eLBati commented Jan 18, 2020

@robyf70 fatto.
Squashed anche

@eLBati
Copy link
Member Author

eLBati commented Jan 18, 2020

/ocabot merge

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-1557-by-eLBati-bump-no, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Jan 18, 2020
Signed-off-by eLBati
@OCA-git-bot OCA-git-bot merged commit c96fa97 into OCA:12.0 Jan 18, 2020
@OCA-git-bot
Copy link
Contributor

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

@eLBati
Copy link
Member Author

eLBati commented Jan 20, 2020

pedro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants