profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/JordiBForgeFlow/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Jordi Ballester Alomar JordiBForgeFlow FORGEFLOW S.L. Barcelona, Spain http://www.forgeflow.com

AaronHForgeFlow/OpenUpgrade 0

Open source upgrade path for Odoo/OpenERP

etobella/workflow 0

Management of generic workflows in odoo

ForgeFlow/account-taxjar 0

Account Taxjar

ForgeFlow/odoo-training-tr 0

Odoo Training TR

JordiBForgeFlow/bank-statement-reconcile 0

Odoo Bank Statements Tasks (reconciliation, completion,...)

JordiBForgeFlow/l10n-spain 0

Odoo Spain Localization

JordiBForgeFlow/maintainer-tools 0

Odoo Maintainers Tools for OCA members which evaluate and maintain repositories.

pull request commentOCA/purchase-workflow

[13.0][add][purchase_order_line_schedule]

@clementmbr Although I love this approach I got some push back from the project I intended to put it in, and finally the PR has been a bit abandoned from my side. But hey, feel free to review, use it and provide your feedback. I am proud of this baby and I thinl that it is a good step forward in the proper scheduling of receptions.

JordiBForgeFlow

comment created time in a day

pull request commentOCA/purchase-workflow

[13.0][ADD] purchase_order_supplierinfo_update: new module

To review @ForgeFlow

ernestotejeda

comment created time in a day

Pull request review commentOCA/purchase-workflow

[13.0][ADD] purchase_order_supplierinfo_update: new module

+# Copyright 2021 Tecnativa - Ernesto Tejeda+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).++from odoo import api, models+++class PurchaseOrder(models.Model):+    _inherit = "purchase.order"++    def write(self, vals):+        res = super().write(vals)+        if vals.get("state", "") in ["purchase", "done"]:+            self.mapped("order_line").update_supplierinfo_prince()

prince --> price

ernestotejeda

comment created time in a day

PullRequestReviewEvent

issue commentOCA/partner-contact

[partner_affiliate] VAT and address are hidden for the Affiliate companies

Proposed PR in https://github.com/OCA/partner-contact/pull/1136

JordiBForgeFlow

comment created time in a day

issue closedOCA/partner-contact

[partner_affiliate] VAT and address are hidden for the Affiliate companies

The module 'partner_affiliate' allows to separate between contacts in a company, and affiliate companies. That's good. However once you assign a parent company to an affiliate, some of the fields in the affiliate, like the address, VAT, become read-only. See for example https://github.com/odoo/odoo/blob/14.0/odoo/addons/base/views/res_partner_views.xml#L202

Obviously this is not what we would expect, as an affiliate, being an business entity, will in most of the cases have a distinct address and possibly VAT.

Alternatively one could use module 'partner_group' https://github.com/OCA/partner-contact/tree/12.0/partner_group but this module, as not using the 'parent_id' will not provide the aggregation capabilities provided by partner_affiliate.

So, in my opinion we must remove changes such as https://github.com/odoo/odoo/blob/14.0/odoo/addons/base/views/res_partner_views.xml#L202 to limit them only when the partner is not a company.

@StephanRozendaal @alexandreduf what would be your opinion?

closed time in a day

JordiBForgeFlow

pull request commentOCA/account-financial-tools

[14.0][IMP] account_asset_management: hook for asset vals

/ocabot merge minor

MiquelRForgeFlow

comment created time in a day

pull request commentOCA/bank-statement-import

[12.0][IMP] account_bank_statement_import_online_transferwise: support SCA

@FerranCforgeFlow can you review?

alexey-pelykh

comment created time in 2 days

pull request commentOCA/account-invoicing

[13.0][IMP]sale_line_refund_to_invoice_qty: do not use negative booleans

@sbidoul can you help me here perhaps? I am PSC of accounting, but I cannot merge PR's in this repo due to this error "The merge process could not start, because of exception 403 Must have admin access to view repository collaborators"

AaronHForgeFlow

comment created time in 2 days

issue openedOCA/stock-logistics-warehouse

[RFC][14.0][stock_location_warehouse]

It is often necessary to know what is the warehouse associated with a location. Currently this connection does not exist and has to be calculated.

This new module would add the field 'warehouse_id' to the stock.location model as a computed, stored field. The computed field would be recomputed with an @api.depends('parent_id'), so that every time that the parent location changes, the warehouse is recomputed. But not only would the warehouse of the location that is changing be recomputed. Also upon recomputation all the child locations below the location that is changing would need to see their warehouse_id field recomputed.

@ForgeFlow

created time in 3 days

push eventForgeFlow/partner-contact

Jordi Ballester

commit sha bb98384bddb539f907f7dfd5ed6da41d43745350

[partner_affiliate][imp] do not change the address of the affiliate when you set up the parent partner.

view details

push time in 3 days

create barnchOCA/partner-contact

branch : 13.0-partner_affiliate

created branch time in 3 days

PR opened OCA/partner-contact

[partner_affiliate][imp] modify partner views to allow you to edit the address and vat of an affiliate.

Currently the address and VAT fields are read-only by the time you choose a parent. That makes sense when you are linking a contact to a parent company, but not when you are linking a company to a parent company.

+29 -0

0 comment

1 changed file

pr created time in 3 days

create barnchForgeFlow/partner-contact

branch : 13.0-partner_affiliate

created branch time in 3 days

PullRequestReviewEvent

pull request commentOCA/web

[FIX] web_widget_bokeh_chart: remove custom-bokeh.scss

:+1: Works for us!

albariera

comment created time in 4 days

push eventForgeFlow/manufacture

Jordi Ballester

commit sha 37bf89db341c87e8256a7a22667ee0cc5394227b

[repair_vendor_refund][add]

view details

push time in 8 days

push eventForgeFlow/manufacture

Jordi Ballester

commit sha 6a27a7e3914aab4ccaddaf589827f3dcf22ad168

[repair_vendor_refund][add]

view details

push time in 8 days

pull request commentOCA/web

[14.0] [MIG] web_widget_mpld3_chart

@pedrobaeza I lost my superpowers. Can you merge?

GenisPForgeFlow

comment created time in 8 days

pull request commentOCA/web

[14.0] [MIG] web_widget_mpld3_chart

/ocabot merge nobump

GenisPForgeFlow

comment created time in 8 days

pull request commentOCA/account-invoicing

[13.0][ADD] account_move_tier_validation_approver

/ocabot merge nobump

MateuGForgeFlow

comment created time in 8 days

PR opened OCA/manufacture

[14.0][repair_vendor_refund]

This module allows to request a refund from the vendor for the components / work that has been included in a repair.

+1096 -0

0 comment

24 changed files

pr created time in 10 days

create barnchForgeFlow/manufacture

branch : 14.0-repair_vendor_refund

created branch time in 10 days

issue commentOCA/partner-contact

[partner_affiliate] VAT and address are hidden for the Affiliate companies

@NL66278 having the parent_id introduces the possibility to aggregate numbers up the hierarchy, which is very interesting. To me what I described looks like a limitation of the partner_affiliate

JordiBForgeFlow

comment created time in 10 days

issue commentOCA/partner-contact

[partner_affiliate] VAT and address are hidden for the Affiliate companies

@sergio-teruel you were involved in the 'partner_group' module. Perhaps you have an opinion on this matter.

JordiBForgeFlow

comment created time in 10 days

issue openedOCA/partner-contact

[partner_affiliate] VAT and address are hidden for the Affiliate companies

The module 'partner_affiliate' allows to separate between contacts in a company, and affiliate companies. That's good. However once you assign a parent company to an affiliate, some of the fields in the affiliate, like the address, VAT, become read-only. See for example https://github.com/odoo/odoo/blob/14.0/odoo/addons/base/views/res_partner_views.xml#L202

Obviously this is not what we would expect, as an affiliate, being an business entity, will in most of the cases have a distinct address and possibly VAT.

Alternatively one could use module 'partner_group' https://github.com/OCA/partner-contact/tree/12.0/partner_group but this module, as not using the 'parent_id' will not provide the aggregation capabilities provided by partner_affiliate.

So, in my opinion we must remove changes such as https://github.com/odoo/odoo/blob/14.0/odoo/addons/base/views/res_partner_views.xml#L202 to limit them only when the partner is not a company.

@StephanRozendaal @alexandreduf what would be your opinion?

created time in 10 days

issue commentOCA/operating-unit

[RFC] Purchase Operating Unit depends on Stock Module but Purchase does not

@gdgellatly Works for me.

gdgellatly

comment created time in 11 days

Pull request review commentOCA/currency

[14.0][MIG] account_cryptocurrency

+Configuration+=============++#. For the user that will do the configuration, make sure to activate the+   option 'Show Full Accounting Features' in the user profile.++#. Activate the Multi-Currency in 'Invoicing / Configuration /+   Settings / Multi-Currencies.++#. Create the cryptocurrency in 'Invoicing / Configuration / Accounting /+   Currencies'. Complete the following fields:  'Inventoried', 'Valuation+   Method'. Indicate the desired rounding factor.++#. Add to the cryptocurrency an 'Inventory Account'. This Account must+   indicate in field 'Account Currency' the same currency.++#. Create an Account Journal associated to the payments of the crypto

The account journal should be of type 'Bank'

AdriaGForgeFlow

comment created time in 13 days

PullRequestReviewEvent