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 account next activities csn #30435

Open
wants to merge 11 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@csnauwaert
Copy link
Contributor

csnauwaert commented Jan 22, 2019

Task: 1911260

@qdp-odoo
Copy link
Contributor

qdp-odoo left a comment

the activity widget should be in community

addons/account/views/account_journal_dashboard_view.xml Outdated
@@ -20,13 +22,17 @@
<div class="row">
<t t-if="(journal_type == 'bank' || journal_type == 'cash')" t-call="JournalBodyBankCash"/>
<t t-if="journal_type == 'sale' || journal_type == 'purchase'" t-call="JournalBodySalePurchase"/>
<t t-if="journal_type == 'general' || journal_type == 'situation'">
<t t-if="journal_type == 'situation'">

This comment has been minimized.

@qdp-odoo

qdp-odoo Jan 23, 2019

Contributor

actually, journal_type == situation doesn't exist anymore...

<div class="col-6 o_kanban_primary_left">
<button type="object" name="open_action" context="{'action_name': 'action_move_journal_line', 'view_no_maturity': 1}" class="btn btn-primary">
<span>Journal Entries</span>
</button>
</div>
</t>
<t t-if="journal_type == 'general'">
<div class="col-12 o_kanban_primary_left o_general_activity">

This comment has been minimized.

@qdp-odoo

qdp-odoo Jan 23, 2019

Contributor

I don't like that placeholder...

the kanban box should always display the activities of the journal, and the widget should be defined in community

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Jan 24, 2019

@csnauwaert csnauwaert force-pushed the odoo-dev:master-account-next-activities-csn branch Feb 20, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 20, 2019

@csnauwaert csnauwaert force-pushed the odoo-dev:master-account-next-activities-csn branch Feb 26, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Feb 26, 2019

@qdp-odoo qdp-odoo force-pushed the odoo-dev:master-account-next-activities-csn branch Mar 5, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 5, 2019

@csnauwaert csnauwaert force-pushed the odoo-dev:master-account-next-activities-csn branch Mar 11, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 11, 2019

csnauwaert and others added some commits Dec 13, 2018

[FIX] account: cur. of reconci. suggestion doesn't match the stmt cur.
1) Enable multi-currency.
2) Setup two bank, one in USD, the other in EUR.
3) Leave the company currency to USD.
4) Create an invoice for a partner targeting the EUR bank and
   validate it.
5) Create a bank statement for the EUR bank amounting the value
   of the invoice but don't reconcile it yet.
6) Create a payment for the partener amounting the value of the
   invoice, targeting the EUR bank and validate it.
7) Come back to the bank statement and reconcile it.
8) The payment is proposed but the value shown is in USD (incorrect)
   whereas the currency shown is EUR (correct)

opw-1937202

@csnauwaert csnauwaert force-pushed the odoo-dev:master-account-next-activities-csn branch to 05532e5 Mar 11, 2019

@robodoo robodoo added CI 🤖 and removed CI 🤖 labels Mar 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.