Comments (29)
Hi @pedrobaeza @jbeficent are you happy for me to migrate this whole repo and commit to review (or at least helping me find a reviewer)? I don't really use it, but I think I'd like to, so would be a straight port unless you have specific comments. Also any other warnings based on current open PR's and issues.
Obviously 1 PR per module, all done by Nov 30 maybe all done next week.
I've had a quick look, there is not much work there really for a straight port. Probably less than a day, a lot of it won't even need a single edit, mostly mangling with git.
from operating-unit.
Thanks for the initiative, Graeme. Of course it's very welcome. The coordinator of this repository is @elicoidal .
from operating-unit.
Ah, no sorry, my fault. It's @jbeficent indeed.
from operating-unit.
:)
from operating-unit.
Hi @gdgellatly, you are very welcome to collaborate to this repo. We will review your PRs and we will try to get more reviewers. Besides It'd be very nice if you review some of the opened PRs in v10
from operating-unit.
@aheficent Of course I will review. There seems some ready to merge, others under discussion. Nothing really left out in the cold needing review. But anything pinged at me I will always review, or just label them as needs review and I'll check back as I get through things.
from operating-unit.
@gdgellatly Thank you. I will label the PRs that are ready to review.
from operating-unit.
@aheficent OK I have migrated everything, I'll just make PR's as they merge as they are all stacked, so when there are changes I'll need to update everything above. It's all at https://github.com/gdgellatly/operating-unit - I did sale_stock_operating_unit last.
After some testing, the repo doesn't quite meet my use case, so I'll probably start something a bit more radical on my own, but I like the concept. Its just too restrictive that each operating unit must have its own warehouses and locations, but was worthwhile getting through the code.
For the most part I can summarize the changes as
- remove now extraneous header and update version
- ensure operating_unit_id is positioned before company in all view where possible
- fix constraints to raise ValidationError rather than UserError
- fix constraints to not constrain related readonly fields
- neaten up most error messages
- don't store fields for no reason
- add some indexes where appropriate (I probably missed a couple here)
- fix some tests that were pulling ids from different models to what they were testing, a few other general test fixups
- where there were obvious bugs (e.g. super call to different function) or performance refactorings I generally did them.
from operating-unit.
@gdgellatly Perhaps the restriction that warehouses and locations should be owned by an operating unit should be relaxed, so that only in the case where you expect a separate balance sheet per operating unit should this restriction be put in place.
from operating-unit.
@gdgellatly In general the issue is that as soon as you use real time inventory accounting, every time you ship a product from a warehouse it is going to hit a COGS account. So how can you decide what operating unit should be attributed for that cost?
from operating-unit.
@gdgellatly thank you for your work, I'll review when creating the PR's to OCA
from operating-unit.
@jbeficent Indeed those are questions I need to answer, I didn't say it was easy. But if I take a logical look at it, even in a self balancing environment I think it can be done.
The COGS case is simple under AngloSaxon as it is accrued at time of sale, and as such easy to apply.
But there is a more general case, beyond stock for difficult to apply items, e.g. Finance and Intangible type items. As I say I'm not sure yet, I have some ideas which I want to test.
From a balance sheet perspective there are always items that cannot be specifically assigned to an OU, e.g. A trademark, or a general bond issue etc but
from operating-unit.
@gdgellatly For trademark, etc. you can have a "corporate" OU.
from operating-unit.
Hello, is there any time estimate for when these PRs will be merged?
from operating-unit.
No estimate. People migrate them to v11 as they need them.
from operating-unit.
Yes but @gdgellatly made a bunch of PRs and states the work is done but awaiting for revision. Need testers? We can do the testing if required
from operating-unit.
@edsersolis Testing would indeed help a lot to get them merged. Please ping me on each PR as you have reviewed.
from operating-unit.
@edsersolis @jbeficent realistically once #99 is merged, and it is mergeable now (all reviews done etc) I can clean up the rest, but no point testing the others yet as we ended up having to change quite a bit.
from operating-unit.
@gdgellatly #99 is now merged. Feel free to propose other PR's. Thank you for the great contribution!
from operating-unit.
@jbeficent the remember to check the mark on the main comment π
from operating-unit.
Hi all. We are new here. We found this modules and could need them in a project. Is there any roadmap on migration to v11?. Regards.
from operating-unit.
@jbeficent #100 has been ready for a few weeks. I've been without reliable internet the last few weeks so will catch up on the rest later.
@marnicodigital its a volunteer community. No one tells anyone to or stops anyone from migrating or reviewing proposals. So there is no roadmap. It is as/when people commit their time and a lot of us are involved in many OCA projects at once. Because the modules are all interdependent, I'm just migrating them as each one is approved and porting the improvements from the review process.
from operating-unit.
We have installed the base module, checked the Multi Operating Unit checkbox in the user's permissions, but it gives us a read permission error. We can create records without problem. We do not even see the operational unit created by default in the installation (OU1). The core is OCB.
We are trying to simulate independent physical stores with this addon, I do not know if apart from the above we are on the wrong way.
from operating-unit.
@meigallodixital Can you provide more info? Where do you get the error in the Operating unit records? Maybe in the Sales team? Have you assigned the problematic user to the operating units in the user form view?
from operating-unit.
@aheficent At Settings -> User & Companies -> Operating Units
We have only installed: operating_unit
I attach screenshot: https://snag.gy/zFVfgi.jpg
from operating-unit.
@meigallodixital Please follow up this issue I created for this: #129 This issue is for sharing the status of the migration only
from operating-unit.
@pedrobaeza please check in the list account_operating_unit - By @grindtildeath - #140
from operating-unit.
res_partner_operating_unit #324
from operating-unit.
There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.
from operating-unit.
Related Issues (20)
- Operating unit for Point Of Sale HOT 2
- how i can transfer products between different OU 2 location or warehouse HOT 6
- Inventory adjustment and operating unit / Analytic account HOT 2
- Operating Unit overwrite incorrect in purchase operating unit - uses previous operating unit HOT 5
- journal Sequences error in vendor payment (account_operating_unit) HOT 2
- [14.0] account_operating_unit compatibility with sale_advance_payment HOT 4
- [14.0] operating_unit - Default operating unit for new public user template ignored HOT 2
- stock_account_operating_unit -Account entry lines bug HOT 5
- [14.0] operating_unit - access restrictions is not applied
- repo readme.md HOT 6
- [12.0] Compatibility with account_payment_order HOT 1
- Migration to version 16.0 HOT 11
- Configure Document Layout in Settings Error HOT 1
- User access not working for reports ? HOT 4
- [12.0,14.0] product_operating_unit: change default behaviour of 'product.template'._get_default_category_id() HOT 1
- Default branch in operating-unit repo HOT 8
- account_operating_unit: inter ou balance lines are displayed in the invoice HOT 3
- 16.0-operating_unit: All OU addes to res users when groups_id change. HOT 1
- Migration to version 17.0 HOT 12
- [17.0][RFC] Challenge: replace operating units by Odoo branches HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
π Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. πππ
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google β€οΈ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from operating-unit.