indeed but hopefully unnecessary).
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
web: http://therp.nl
___
Mailing list: https://launchpad.net/~openerp
to wait for the OpenID code from OpenERP to show
up and try to hook into that as it probably addresses many of the same
issues.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
web: http
in the OpenERP community as well as supporting OpenERP SA in
their business model.
On behalf of the Therp team,
Stefan Rijnhart
[1] http://www.openerp.com/forum/topic26369.html
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0
-1283369.html?page=2
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
web: http://therp.nl
___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp
they might have been hinting at OpenERP's license policy with
the third reason, but the second reason seems more vital to me.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
web: http://therp.nl
:
https://code.launchpad.net/~openerp-community/openobject-addons/extra-trunk-esale_joomla
Daniel Stenlöv has been updating it over the last couple of days.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0
weekend.
/Best wishes, Daniel
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net
, it
is a problem with the OpenERP module format in that it does not have
version control. It hurts the extra-addons *and* the apps site.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https
often...
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp
issue, can you give a snippet of the code
that illustrates your problem?
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
self.pool.get('res.partner').write(cr, uid, {'name': 'New name'}
Would not want to befog the kids trying this out at home, so rather
self.pool.get('res.partner').write(cr, uid, 1, {'name': 'New name'})
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en
ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
shortcuts for [S]ave, [E]dit,
[C]reate and [D]iscard has just been merged into the web client yesterday.
https://code.launchpad.net/~therp-nl/openerp-web/openerp-web_accesskeys/+merge/118515
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail
client.
Instead of trying to keep the GTK client on life support, I would rather
focus on making the Web client more data entry friendly (main issue is
lack of keyboard shortcuts for most operations)
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en
.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp-community
Post
2012/11/5 Carlos Liébana carlos.lieb...@factorlibre.com
mailto:carlos.lieb...@factorlibre.com
Hello,
I'm founding a lot of bugs with fix released but not being
merged in the main branch. This is leading us and I suppose
the rest of companies to have
On 11/20/2012 11:06 PM, Sébastien LANGE (SYLEAM) wrote:
Hi Joel,
Great initiative and thank you for all you effort.
I prefer person instead of team and it's good idea to vote for having
a new person in OpenERP Community Reviewer.
Voting is a good idea. Can we have a mailinglist for the team
On 11/21/2012 11:19 AM, Alexis de Lattre wrote:
OpenERP S.A. says that they are aware of this issue :
We realize that up to now OpenERP was sending within a few days a
patch to the customer or partner, but was taking time to merge it into
the latest stable version of OpenERP. We have
the release of 7.0.
Best regards,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https
/Pympler/library/process.html
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https
On 12/13/2012 01:26 AM, Fabrice Henrion wrote:
I'm not an expert but it seems like psutils runs on Solaris:
http://knackered.org/angus/psutils/
Ah, but these are *Postscript* utils, not *process* utils. For the
record, the library that I linked to myself does not seem to allow you
to
be much of a
challenge when migrating to 7.1 eventually either through the
OpenUpgrade project or the paid migration service offered by OpenERP SA
but in the latter case contact OpenERP SA to be sure.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en
On 01/10/2013 07:40 PM, Mario Arias wrote:
At least for date fields, consensus was that system should record the
date as seen by the user creating the record, using the new
context_today method...
- Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp
not rely on it as a company.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net
to start, I suppose docs?
Hi,
The autodoc of the API is back again at
https://openupgrade-server.readthedocs.org/en/latest/API.html
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
On 04/01/2013 12:14 AM, Etienne Hirt wrote:
While using 6.1 mail_message to_email function I found that it does not
cope with all the possible names in the emaillist.
I therefore enhanced the regexp in function to remove also , () and
even if allowed '.
The full version is found in
://twitter.com/rvalyi http://twitter.com/#%21/rvalyi
+55 21 2516 2954
www.akretion.com http://www.akretion.com/
On Tue, Apr 2, 2013 at 5:25 AM, Stefan Rijnhart ste...@therp.nl
mailto:ste...@therp.nl wrote:
On 04/01/2013 12:14 AM, Etienne Hirt wrote:
While using 6.1 mail_message
be
for people who suggest any changes to help reviewing other proposals.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
opinion on this.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net
,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp-community
Post
Hi,
as discussed on this list, the 6.1 series have been opened on the
backports projects. The branches from our old project have now been
pushed and linked to the series. Two new FAQs have been created
mentioning the origin of the 6.1 branches and the functional changes in
them so that you
unsupported branch in that case, to be honest.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
On 04/12/2013 08:40 AM, Mario Arias wrote:
Thanks for your help.
I got farther this time. I could assign the bug to OCB 6.1 and 7.0.
I could not link the branch with the fix to them (I guess because
original fix is applied to trunk, not 6.1 or 7.0)
Should I branch out 6.1 and 7.0, apply
to your merge proposals!
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https
ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community
On 06/13/2013 01:56 PM, Nicolas Bessi wrote:
Hello,
So should we wait 3 or 5 days before fixing a community rule :P ?
Actually we have:
3 in favor of 3 reviewers
1 in favor of 2 reviewers
2 in favor of 3 days
1 in favor of five days
Hi,
For the record, I suggested the number of three
On 08/17/2013 05:32 PM, Maxime Chambreuil wrote:
Not having this field will force people to collaborate and solve the
conflict when discovered.
Not particularly. I remember the first time someone told me that Banking
Addons were incompatible with account_bank_statement_extension. I did
not
On 08/17/2013 06:45 PM, Stefan Rijnhart wrote:
As the defacto maintainer of the Banking Addons, I do not feel forced
to collaborate and integrate with alternative solutions. I might think
it is a good thing if I see added value for my company or my customers
in any way, but I am not forced
Thank you Fabien, for noticing this! And I hope the issue Ronald
signalled can be solved so that we can use the runbot for keeping track
of problems with the OCB branches.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
On 10/27/2013 08:19 AM, Fabien Pinckaers wrote:
Ferdinand,
This is not an issue anymore. The new realease policy fixes this. We
don't do anymore cycles like: develop for 9 months, fix during 3
months, then release.
Now, the trunk is stable. Every new feature in trunk goes to minimum 3
code
On 10/28/2013 09:29 PM, Stefan Rijnhart wrote:
That means that a two way granularity on branch level (splitting up
schema changers from the rest) may good enough to base a
semi-automated feedback process on.
Oh, I mean
may *not be* good enough
On 10/28/2013 08:30 PM, Olivier Dony wrote:
So it's very simple: if the OCB contributors find it useful to have a
faster commit path to the stable branches, then they might find that
respecting the stable policy somehow is worth it. It's an easy way to
give more power to the community.
and if they don't that is fine too. And if we can make it a
little easier for SA without disturbing the OCB collaborative process
too much then we will surely do that.
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail:ste...@therp.nl
tel
On 02/09/2014 11:55 AM, Bidoul, Stéphane wrote:
On Sun, Feb 9, 2014 at 1:26 AM, Georges Racinet graci...@anybox.fr
mailto:graci...@anybox.fr wrote:
There are currently two long broken YML tests in the whole
distribution (in account_payment and in purchase_requisition). I
could not
On 02/24/2014 05:27 PM, Ferdinand Gassauer wrote:
I think in general the code should be 2.6 compatible for.
Agreed. I think it would make sense to follow the OpenERP distribution's
defacto[1] compatibility with Python releases which would include 2.6
for modules compatible with OpenERP 7.0.
On 02/25/2014 03:02 PM, Raphael Valyi wrote:
Hello Stefan
Then eventually, in a pure Launchpad scenario, may be the easiest
would be have an automated replica of OCB v7 back into the main
official project (using automatic replay script that won't fail
because it's just a replica of OCB v7).
On 03/04/2014 07:04 PM, Olivier Dony wrote:
We usually try to follow debian-stable in terms of dependencies, as
illustrated in the attached picture. We've started to use some 2.7
features in the v8/trunk codebase so there's a good chance v8 will
require 2.7.
Thanks for your comments! Seems
On 04/04/2014 11:05 AM, Sebastien Beau wrote:
Hi All,
I always hesitate to move or not on CamelCase for every class in OCA
module.
I prefer CamelCase but a lot of code do not use it.
And sometime not using CamelCase can introduce bug and you have to
rename the class in something different
On 04/23/2014 03:11 PM, Alexandre Fayolle wrote:
Quick question : is it mandatory to have a mail server allowing for the
configuration of a catch all email address in order to get correct
mail exchange and record creation on some mail receptions ?
Hi Alexandre,
no, I don't use catchall
On 16-05-14 13:43, Alejandro Santana wrote:
Also: Arista font is here: http://www.dafont.com/es/arista.font
FYI: the license indicates that the font must be bought for commercial
purposes.
Regards,
Stefn.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en
ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
___
Mailing list: https://launchpad.net/~openerp-community
Post to : openerp-community@lists.launchpad.net
On 05/19/2014 07:38 PM, Nhomar Hernández wrote:
I think the proposal y 1 Organization (organization == Team)/ Several
repositories (repository == project-launchpad).
Then we can follow the Repository under the Organization.
Thanks Nhomar,
the way you describe it, I do agree with
of another more appropriate ?
Regards,
Joël
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
--
Mailing list: https://launchpad.net/~openerp-community
/148245
Feel free to say what needs fixing!
Thank you very much Jay, that is examplary. I will post my review.
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com
this should be a good way of integrating the team : making
review first and then ask to be part of it. Not to put barrier, I know
I can trust you, but more to starts putting some organization. What do
you think ?
Other opinion ?
Best regards,
--
Therp - Maatwerk in open ontwikkeling
Stefan
-tools.
Which option do the community reviewers prefer?
Cheers,
Stefan.
[1] http://bazaar.launchpad.net/~hbrunn/+junk/resource_planning/files
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https
On 12/11/2013 04:59 PM, Joël Grand-Guillaume wrote:
Dear all,
I'm voting in favor of Sandy Carterand Yann Papouin as member of the
reviewer team.
I'm also suggesting Yannick Vaucher as he really invest lots of time
those last month:
+1 for all three of these great contributors.
Cheers,
response!
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp
On 04/28/2014 10:15 PM, Stefan Rijnhart wrote:
beaten to us
has beaten us to it...
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-community-reviewer
More help
+1 for this very active contributor.
On 05/01/2014 10:15 AM, Nicolas Bessi wrote:
Dear community,
I would like to propose Leonardo Pistone as member of community reviewer.
He his an active contributor and reviewer of OpenERP community for
quite a time now
and would be a perfect fit for the
ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
--
Mailing list: https://launchpad.net/~openerp-community-reviewer
Post to : openerp-community-reviewer@lists.launchpad.net
Unsubscribe : https
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-addons/stefan-therp_lp731430-trunk into
lp:openobject-addons.
Requested reviews:
OpenERP Core Team (openerp)
Related bugs:
Bug #731430 in OpenERP Addons: l10n_nl: order in tax report is undefined
https
-community/openobject-client-web/stefan-therp_lp748238-6.0/+merge/57517
Thanks for the review and the merge!
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
web: http://therp.nl
https
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-server/stefan-therp_lp789576 into
lp:openobject-server/6.0.
Requested reviews:
OpenERP Core Team (openerp)
Related bugs:
Bug #789576 in OpenERP Server: [6.0] fields.related override: store function
persists
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-addons/stefan-therp_lp696176 into
lp:openobject-addons/6.0.
Requested reviews:
OpenERP Core Team (openerp)
Related bugs:
Bug #696176 in OpenERP Addons: TypeError: node contents must be a string
when using
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-addons/stefan-therp_lp794450 into
lp:openobject-addons.
Requested reviews:
OpenERP Core Team (openerp)
Related bugs:
Bug #794450 in OpenERP Addons: [users_ldap] Wishlist: allow anonymous bind
https
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-addons/stefan-therp_lp794584 into
lp:openobject-addons.
Requested reviews:
OpenERP Core Team (openerp)
Related bugs:
Bug #794584 in OpenERP Addons: [users_ldap] Wishlist: populate OpenERP user
base from LDAP
resorted to simply replacing a None
value for the bind DN and password with the empty string. You can now
even have a harmless unauthenticated authentication if your ldap server
allows it by only filling in the bind DN ;-)
Cheers,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan
in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
web: http://therp.nl
https://code.launchpad.net/~openerp-community/openobject-addons/stefan-therp_lp794584/+merge/63872
Your team OpenERP Community is subscribed to branch
lp:~openerp
Review: Resubmit
On 06/20/2011 10:21 AM, Stefan Rijnhart (Therp) wrote:
I will therefore reuse this branch to suggest such a breakdown in
separate functions, and leave the added functionality for a separate
module of our own make.
Sounds great, thanks! :-)
Hi Olivier,
this branch now
The proposal to merge
lp:~openerp-community/openobject-addons/stefan-therp_lp794584 into
lp:openobject-addons has been updated.
Description changed to:
This branch breaks down LDAP authentication into a number of atomic functions
so that additional modules can more easily override specific
) 2392 files modified (has conflicts)
https://code.launchpad.net/%7Eopenerp-community/openobject-addons/fix-819242/+merge/69990/+preview-diff/+files/preview.diff
Regards,
Stefan.
--
Therp - Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31
Review: Needs Fixing
Hi Ian,
thanks for resubmitting this. I have not seen this prerequisite branch
functionality of bzr in action before, but it appears to be working pretty well
apart from the phantom addition of 'hr_payroll/i18n/vi.po' that none of us
commited in these branches (but that
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-doc/stefan-therp_lp845135 into
lp:openobject-doc.
Requested reviews:
graylion (graylion)
Els Van Vossel (OpenERP) (evv-openerp)
Related bugs:
Bug #845135 in OpenERP Documentation: Description of Requests form
Stefan Rijnhart (Therp) has proposed merging
lp:~openerp-community/openobject-server/stefan-therp_lp727727-6.1 into
lp:openobject-server.
Requested reviews:
OpenERP Core Team (openerp)
Related bugs:
Bug #727727 in OpenERP Server: Orm.create() should not drop nonexisting
fields in passed
Review: Resubmit
Hi Olivier,
thanks again for the valuable review.
On your suggestion, I have replaced the exception with a warning. I used a
logging.warn instead of a warnings.DeprecationWarning as these cannot easily be
redirected to the application log unless one requires Python 2.7 (with
Hi Lorenzo,
thanks for the merge! Yes, we should definitely merge into extra-6.0. I have
two more fixes to test and upload:
- In the plugin, retrieve language 'iso' column, not 'shortcode' to prevent
'Cannot match language' problem during XML-RPC communication (the plugin
reports language
Bug #949125 status changed in Therp Addons:
New = Fix Released
https://bugs.launchpad.net/therp-addons/+bug/949125
Custom report error when Click on button OpenReport
This bug is linked to #190081.
Custom report error when Click on button OpenReport
- Maatwerk in open ontwikkeling
Stefan Rijnhart - Ontwerp en implementatie
mail: ste...@therp.nl
tel: +31 (0) 614478606
http://therp.nl
https://twitter.com/therp_stefan
https://code.launchpad.net/~openerp-community/openobject-addons/fix-account-trunk-tax-computation-method/+merge/104945
Your team
Review: Abstain
Well this is spellingwise an improvement but judging by the code of the module
it has bigger problems than this. None of the three fields that
_membership_date can deliver share the same 'multi' argument, which neatly
corresponds to the fact that this function only delivers
Hi Tien,
the diff is mangled because of changes in the file mode of all or many files in
the repository. Did you work with the branch on a filesystem that does not
support such file modes or has to make assumptions about them? [1] indicates
that the problem occurs in situations where a Windows
This MP is set to merged but it is not really.
http://bazaar.launchpad.net/~openerp/openobject-server/7.0/revision/4786
--
https://code.launchpad.net/~openerp-community/openobject-server/fix-1094212-multicompany-res_partner/+merge/141936
Your team OpenERP Community is subscribed to branch
The proposal to merge
lp:~openerp-community/openobject-server/fix-1094212-multicompany-res_partner
into lp:openobject-server/7.0 has been updated.
Status: Merged = Needs review
For more details, see:
Review: Needs Fixing
Hi Tran,
thank you for updating the code. I still have some comments:
Indentation still not correct, the lines inside the record tags should be
indented more than those tags.
Account types are now 'noupdate', but so are account templates. You will want
to include these
Review: Approve
Looks good to me! The OpenERP core developers need to take it from here.
--
https://code.launchpad.net/~openerp-community/openobject-addons/gscom-70-l10n-vn/+merge/141526
Your team OpenERP Community is subscribed to branch
Review: Needs Fixing
Hi,
in OpenERP 7.0, css is encoded in addons/web/static/src/css/base.sass and then
compiled to css using Sass. See http://sass-lang.com/tutorial.html. Please
commit the required change to the base.sass and the compiled css file.
--
Review: Needs Fixing
There are already two modules on apps.openerp.com that do this. Not saying that
it is not useful to have one in the community project, but what you can learn
from them is that they make sure to add a ' (copy)' suffix when copying a
product. Presumably the constraint would
Please stop merging so fast.
If this is a submit of the current state of affairs on 7.0, there were all
kinds of problems with it. See my review there. Serpent never addressed my
comments directly but updated the branch periodically. I believe it has
improved in certain aspects, but it was
Review: Needs Fixing
My review is a bit compact but feel free to discuss anything that you disagree
about.
l. 128 e.a. Please replace the whole stringified ID list thing by a regular
many2many field. Adapt onchange_model to return just a list of ids. Replace the
eval'ed string clause in the
To be honest, I was hoping to speed up the process if someone else started to
work on it. The V7 of this branch only fixes the data loss issue and the typos,
so the majority of the issues still stand.
--
Good idea, and thanks again for picking this up!
--
https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619
Your team OpenERP Community is subscribed to branch
lp:~openerp-community/server-env-tools/6.1-mass_editing.
Review: Approve
You're quick! Thanks for the change.
--
https://code.launchpad.net/~eoc/server-env-tools/6.1-mass_editing-fix_bug_1187937/+merge/167643
Your team OpenERP Community is subscribed to branch
lp:~openerp-community/server-env-tools/6.1-mass_editing.
You need to do it yourself and commit the result. Install the sass ruby gems,
then do
/path/to/gems/bin/sass -t expanded base.sass base.css
--
https://code.launchpad.net/~openerp-community/openerp-web/trunk-bug-1107762/+merge/147506
Your team OpenERP Community is subscribed to branch
Pieter, to avoid any confusion:
this is about a branch
lp:~openerp-community/openobject-addons/trunk-addons-community which contained
a very old version of the account_banking module, not about lp:account-banking
itself.
--
Thanks for picking up the changes! I am afraid I forgot to mention another one:
in line 40, you mention that the license should be set to 'AGPL'. Maybe change
this to 'AGPL-3' to reflect that this is the exact entry in the module model's
selection field. Modules with a license literally set to
Review: Approve
Thanks for the changes!
--
https://code.launchpad.net/~openerp-community/openobject-doc/7.0-community-review/+merge/186295
Your team OpenERP Community is subscribed to branch
lp:~openerp-community/openobject-doc/7.0-community-review.
Review: Needs Fixing
I received a resubmit in my mail, meaning that you put the status of this MP to
'Needs review', but the dataloss issue has not yet been resolved.
--
https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619
Your team OpenERP Community is
Hi, I'd still be interested in a response to my comment about UOM.
--
https://code.launchpad.net/~vauxoo/openerp-product-attributes/7.0-prod_vol_weight-dev-humbertoarocha/+merge/189956
Your team OpenERP Community is subscribed to branch
lp:openerp-product-attributes.
I'm proposing the long awaited change (AFAIK) to this branch here:
https://code.launchpad.net/~therp-nl/server-env-tools/6.1-mass_editing-fix_dataloss/+merge/201321
--
https://code.launchpad.net/~openerp-community/server-env-tools/6.1-mass_editing/+merge/161619
Your team OpenERP Community is
1 - 100 of 460 matches
Mail list logo