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

Master pos improve usability mgh #31551

Closed

Conversation

mgh-odoo
Copy link
Contributor

@mgh-odoo mgh-odoo commented Mar 4, 2019

@C3POdoo C3POdoo added the RD research & development, internal work label Mar 4, 2019
@mgh-odoo mgh-odoo force-pushed the master-pos-improve-usability-mgh branch 2 times, most recently from c995d90 to 7b10cab Compare March 4, 2019 07:05
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Mar 4, 2019
@@ -178,7 +179,7 @@ def _balance_check(self):
account = stmt.journal_id.profit_account_id
name = _('Profit')
if not account:
raise UserError(_('There is no account defined on the journal %s for %s involved in a cash difference.') % (stmt.journal_id.name, name))
raise UserError(_('Please go on the "%s" journal and define a %s Account.\nThis account will be used to record the cash diffrence.') % (stmt.journal_id.name, name))
Copy link
Contributor

Choose a reason for hiding this comment

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

difference

@mgh-odoo mgh-odoo force-pushed the master-pos-improve-usability-mgh branch 2 times, most recently from 4bb8afa to 77ff287 Compare March 6, 2019 06:14
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Mar 6, 2019
@mgh-odoo mgh-odoo force-pushed the master-pos-improve-usability-mgh branch from 77ff287 to b03cecf Compare March 6, 2019 06:53
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Mar 6, 2019
- currency symbol added to the account.bank.statement.cashbox form view
- removed message of cash_journal_id from the pos.session form view
- Make Journal field clickable of pos.session form view

Task: #1934784
Closes: odoo#31551
There should be a default gain and default loss account on the cash journal
with the US accounting localization

Task: #1934784
Closes: odoo#31551
should be defined on the POS, not per journal
moved amount_authorized_diff setting on the pos.config

Task: #1934784
Closes: odoo#31551
@mgh-odoo mgh-odoo force-pushed the master-pos-improve-usability-mgh branch from b03cecf to 9073afd Compare March 25, 2019 11:43
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Mar 25, 2019
@mgh-odoo
Copy link
Contributor Author

Merged here #33948

@mgh-odoo mgh-odoo closed this Jul 15, 2019
@mgh-odoo mgh-odoo deleted the master-pos-improve-usability-mgh branch July 15, 2019 06:11
@robodoo robodoo added closed 💔 and removed CI 🤖 Robodoo has seen passing statuses labels Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
RD research & development, internal work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants