IRC logs of #tryton for Wednesday, 2017-03-29

chat.freenode.net #tryton log beginning Wed Mar 29 00:00:01 CEST 2017
2017-03-29 00:20 -!- thaneor(~ldlc6@r167-56-51-129.dialup.adsl.anteldata.net.uy) has joined #tryton
2017-03-29 00:25 -!- smarro(~sebastian@cust-138-45-13-200.trynet.com.ar) has joined #tryton
2017-03-29 00:27 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 01:39 -!- cedk(~ced@gentoo/developer/cedk) has joined #tryton
2017-03-29 01:49 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 02:52 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 03:35 -!- smarro(~sebastian@cust-138-45-13-200.trynet.com.ar) has joined #tryton
2017-03-29 04:00 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 04:28 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 04:39 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 06:03 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 06:14 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 06:58 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 07:51 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 08:02 -!- dj_xatra(~dj_xatra@217.166.83.130) has joined #tryton
2017-03-29 08:05 -!- morfeokmg(~morfeokmg@201.103.192.201) has joined #tryton
2017-03-29 08:34 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 08:42 -!- Timitos(~kpreisler@host-88-217-184-172.customer.m-online.net) has joined #tryton
2017-03-29 08:45 -!- rpit(~rpit@2a02:908:e672:7480:56ee:75ff:fe0d:d3c7) has joined #tryton
2017-03-29 08:49 -!- rpit(~rpit@2a02:908:e672:7480:56ee:75ff:fe0d:d3c7) has joined #tryton
2017-03-29 08:51 -!- cedk(~ced@gentoo/developer/cedk) has joined #tryton
2017-03-29 09:11 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 09:17 -!- mrichez(~smuxi@mail.saluc.com) has joined #tryton
2017-03-29 09:28 <pokoli> cedk: I'm reading your messages now
2017-03-29 09:32 <pokoli> cedk: I don't mather about the time (we have this patch already applied on test environments) but I don't see what else must be done in the patch
2017-03-29 09:46 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 09:48 <cedk> pokoli: that's the problem, it is not about "what est must be done"
2017-03-29 09:49 <cedk> pokoli: it is about finding the right solution
2017-03-29 09:49 <cedk> pokoli: and so what is exhausting is to have to tell many times, it is not the right solution
2017-03-29 09:51 <cedk> I have already reduced my review job for others because when after 10 reviews there are still the same issue, it is a waste of time
2017-03-29 09:51 -!- rpit(~rpit@2a02:908:e672:7480:56ee:75ff:fe0d:d3c7) has joined #tryton
2017-03-29 09:54 <pokoli> cedk: sometimes review comments are complex to understand
2017-03-29 09:55 <pokoli> cedk: and then you don't realese what's the rigth (or best) solution until the third comment
2017-03-29 09:55 <pokoli> cedk: indeed, it's a litlle bit exhausiting for the other side also
2017-03-29 09:56 <pokoli> cedk: but at I think that after the time spent we reach the goal (to have the right solution)
2017-03-29 09:57 <cedk> pokoli: I think I do not care to hear about that.
2017-03-29 09:57 <cedk> pokoli: then I will stop even sooner reviewing, especially patch that I do not really care
2017-03-29 09:59 <pokoli> cedk: you are free to do whatever you want with your time
2017-03-29 10:00 <pokoli> cedk: but what I want to note is that we all have to put from our side
2017-03-29 10:06 <cedk> As I have the feeling that people relies on my reviews to get things done properly, I will just stop reviewing after the 5th set
2017-03-29 10:09 <pokoli> cedk: so you mean that you will "discard" patches after the 5th set?
2017-03-29 10:11 <cedk> just ignore
2017-03-29 10:11 <pokoli> then review will be there forever without feedback and never commited
2017-03-29 10:12 <cedk> yep
2017-03-29 10:12 <pokoli> and will produce the feeling of: "Nobody cares about my patches"
2017-03-29 10:13 <cedk> I think it is the only solution for me to stay sane and push pressure to contributor to think twice before submitting a patch
2017-03-29 10:14 <pokoli> cedk: yeah, it's important for you to stay sane :)
2017-03-29 10:14 <pokoli> cedk: ok, but maybe it's worth to send an email to dev ML explaining the situation
2017-03-29 10:21 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 10:30 -!- nicoe(~nicoe@host-85-201-184-151.dynamic.voo.be) has joined #tryton
2017-03-29 10:38 <mrichez> cedk: hi Ced, following your discuss on reviews : so what's the best way, for beginners to have advices on their way of coding in tryton? Trying to post a patch is a way of learning, no ? In the end, we could be afraid to post a patch because we are not expert but good will...
2017-03-29 10:40 <cedk> mrichez: I do not know, I have no solution
2017-03-29 10:40 <cedk> but an initial problem is there is not enough people reviewing
2017-03-29 10:42 <mrichez> cedk: maybe but we need some tryton level in coding
2017-03-29 10:42 <pokoli> mrichez: for sure it's not good to stop doing things because you are afraid, so I will say ask whatever doubts you have and we will try to answer them
2017-03-29 10:43 <pokoli> cedk: agree with mrichez it's not easy to review until you have some background. Indeed, I've been reviewing for a lot of time and I'm still learing things everyday
2017-03-29 10:53 <pokoli> cedk: don't now if it will help if I (or somebody else) give a first review on patches and after all the patches are review, you give the final review
2017-03-29 10:56 <tbruyere_> I'm not often on IRC but I find the discussion important: my dream would be to become "reviewer" ... But currently I just have time to read and monitor the "patch" and "issue". My level in python and tryton is not yet enough to allow me remarks in other codes. Unfortunately it is only excuses for not investing in the project really. I think this is the problem of many ....
2017-03-29 11:03 <cedk> pokoli: of course it will help, if there were a initial review
2017-03-29 11:03 <cedk> pokoli: but as far as we do not have enough reviewers, I do not see how it could work
2017-03-29 11:04 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 11:05 <pokoli> cedk: correctme if I'm wrong, but currently you and me are the ones we are normally reviewing
2017-03-29 11:06 <pokoli> cedk: so if we can provide some workflow to improve stay saner better. And if then more people join I'm sure we can find the space on the workflow
2017-03-29 11:10 -!- cedk(~ced@gentoo/developer/cedk) has joined #tryton
2017-03-29 11:12 -!- nicoe(~nicoe@85.201.184.151) has joined #tryton
2017-03-29 11:39 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 11:49 -!- csotelo(~csotelo@2001:1388:49c4:26:3b38:fb54:94df:ddbc) has joined #tryton
2017-03-29 12:22 -!- thaneor(~ldlc6@r179-25-33-251.dialup.adsl.anteldata.net.uy) has joined #tryton
2017-03-29 12:42 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 13:03 -!- smarro(~sebastian@cust-138-45-13-200.trynet.com.ar) has joined #tryton
2017-03-29 13:04 -!- mariomop(~quassel@186.125.16.84) has joined #tryton
2017-03-29 13:47 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 14:13 -!- smarro(~sebastian@cust-219-46-13-200.trynet.com.ar) has joined #tryton
2017-03-29 14:24 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 14:34 <pokoli> cedk: about the dict patch, I imagine you propse to add a json adapter using pyscopg2 extras but I have some doubts:
2017-03-29 14:35 <pokoli> jsonb is only supported since 2.5.4 (but we required 2.5), should I update the require version or add conditional support?
2017-03-29 15:11 <cedk> pokoli: precesly what I do not want to do, thinking about that now
2017-03-29 15:16 <pokoli> cedk: ok, sorry and take your time
2017-03-29 15:18 <cedk> pokoli: I will talk about that in 1 month
2017-03-29 15:19 <pokoli> cedk: are you going to have holidays untill the release?
2017-03-29 15:25 <cedk> no
2017-03-29 15:28 <pokoli> cedk: I updated the review but please don't look at it until we talk about it :)
2017-03-29 15:31 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 16:12 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 17:20 -!- pijuli(~marc@170.red-80-28-119.adsl.static.ccgg.telefonica.net) has joined #tryton
2017-03-29 17:20 -!- pijuli(~marc@170.red-80-28-119.adsl.static.ccgg.telefonica.net) has left #tryton
2017-03-29 17:21 -!- marcP(~marc@170.red-80-28-119.adsl.static.ccgg.telefonica.net) has joined #tryton
2017-03-29 17:23 <marcP> Hi everyone, every day question before going home is here :P This time is about changing a field marked as 'required=True' of product from a __setup__. I know how to change states and so, but not required flag. Any clue or example? I've been searching for it for a while
2017-03-29 17:37 <pokoli> marcP: indeed why try to don't add to much required fields to avoid this issues
2017-03-29 17:37 <pokoli> marcP: which field are we talking about?
2017-03-29 17:40 <marcP> pokoli: list_price, cost_price, cost_price_method and default_uom from product.template. By default are required but I won't need them ^^
2017-03-29 17:41 <pokoli> marcP: and you can not hide and enter a default value for it?
2017-03-29 17:43 <marcP> pokoli: yes, I can do that, it's an option :) just by curiosity, isn't it easier to change the requirement?
2017-03-29 17:43 <pokoli> marcP: have you tested setting required=False to the field and dropping the not null contraint?
2017-03-29 17:44 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 17:45 <marcP> do you mean this? : cls.cost_price.required=False . If you do yes, I've tried. But didn't change the constrain. Anyway the form is still requiring me the field
2017-03-29 17:49 <pokoli> marcP: how have you done it?
2017-03-29 17:49 <pokoli> marcP: if you don't change the not null constraint the field will be required on the database
2017-03-29 17:54 -!- smarro(~sebastian@cust-138-45-13-200.trynet.com.ar) has joined #tryton
2017-03-29 18:08 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 18:10 -!- cryptic(~cryptic@67-8-35-31.res.bhn.net) has joined #tryton
2017-03-29 18:16 -!- smarro(~sebastian@200.13.45.138) has joined #tryton
2017-03-29 18:19 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 18:50 -!- smarro(~sebastian@cust-138-45-13-200.trynet.com.ar) has joined #tryton
2017-03-29 18:52 -!- Telesight(~anthony@4dae0c97.ftth.telfortglasvezel.nl) has joined #tryton
2017-03-29 19:01 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 19:32 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 20:16 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 20:29 -!- smarro(~sebastian@200.13.46.123) has joined #tryton
2017-03-29 20:31 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 20:57 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 21:11 -!- lukio(~lukio@181.47.220.45) has joined #tryton
2017-03-29 21:28 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 21:33 -!- JosDzG(~Thunderbi@189.250.83.57) has joined #tryton
2017-03-29 21:35 -!- rpit(~rpit@aftr-37-24-149-139.unity-media.net) has joined #tryton
2017-03-29 21:44 -!- nicoe(~nicoe@2a02:a03f:30b2:3c00:ee55:f9ff:fe7b:f7ac) has joined #tryton
2017-03-29 21:56 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 22:24 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton
2017-03-29 22:26 -!- JosDzG(~Thunderbi@187.171.237.107) has joined #tryton
2017-03-29 22:40 -!- lukio(~lukio@181.47.220.45) has left #tryton
2017-03-29 23:00 -!- trilliji(~trilliji@cpe-67-255-39-42.stny.res.rr.com) has joined #tryton

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!