Difference between revisions of "IDempiere/FullMeeting20141105"
CarlosRuiz (talk | contribs) (full meeting) |
CarlosRuiz (talk | contribs) (drop JIRA notifications from log) |
||
| Line 11: | Line 11: | ||
'''''CarlosRuiz''''': merging zk7 version<br> | '''''CarlosRuiz''''': merging zk7 version<br> | ||
'''''Deepak''''': Morning<br> | '''''Deepak''''': Morning<br> | ||
| − | |||
'''''Deepak''''': Are you merging zk7 to development?<br> | '''''Deepak''''': Are you merging zk7 to development?<br> | ||
| − | |||
'''''aguerra''''': Buenos dias, good morning!!!!<br> | '''''aguerra''''': Buenos dias, good morning!!!!<br> | ||
| − | |||
| − | |||
'''''CarlosRuiz''''': Hi Deepak - Hi Alejandro<br> | '''''CarlosRuiz''''': Hi Deepak - Hi Alejandro<br> | ||
'''''CarlosRuiz''''': not yet Deepak - I'm merging dev into zk7 first and integrating fixes from hiep<br> | '''''CarlosRuiz''''': not yet Deepak - I'm merging dev into zk7 first and integrating fixes from hiep<br> | ||
| Line 22: | Line 18: | ||
'''''Deepak''''': I am testing import inventory with cost update having ASI level costing<br> | '''''Deepak''''': I am testing import inventory with cost update having ASI level costing<br> | ||
'''''Deepak''''': And found one bug and looks may have more on my way as this not looks tested<br> | '''''Deepak''''': And found one bug and looks may have more on my way as this not looks tested<br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
'''''hieplq''''': hi Deepak i not sure i understand your requiremnet relate to question in prev meeting. "How can we create pull request for single commit?" maybe two branch as tbayen suggrest and graft command of hg can help<br> | '''''hieplq''''': hi Deepak i not sure i understand your requiremnet relate to question in prev meeting. "How can we create pull request for single commit?" maybe two branch as tbayen suggrest and graft command of hg can help<br> | ||
| − | |||
| − | |||
'''''Deepak''''': Thanks hieplq, I yet need to try that<br> | '''''Deepak''''': Thanks hieplq, I yet need to try that<br> | ||
'''''Deepak''''': I have some ticket to push in dev, So will do that<br> | '''''Deepak''''': I have some ticket to push in dev, So will do that<br> | ||
'''''hieplq''''': it's easy to use TortoiseHg. i just know it when i bring some my local patch into zk7<br> | '''''hieplq''''': it's easy to use TortoiseHg. i just know it when i bring some my local patch into zk7<br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
'''''TurkerTunali''''': goodmorning<br> | '''''TurkerTunali''''': goodmorning<br> | ||
'''''TurkerTunali''''': it is 15:00 here<br> | '''''TurkerTunali''''': it is 15:00 here<br> | ||
'''''CarlosRuiz''''': Hi TurkerTunali <br> | '''''CarlosRuiz''''': Hi TurkerTunali <br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
'''''tbayen''''': Daarestiet<br> | '''''tbayen''''': Daarestiet<br> | ||
'''''TurkerTunali''''': I've accidentally right clicked and found 3 new messages to translate<br> | '''''TurkerTunali''''': I've accidentally right clicked and found 3 new messages to translate<br> | ||
| Line 54: | Line 34: | ||
'''''TurkerTunali''''': no no, I am fine, thank you<br> | '''''TurkerTunali''''': no no, I am fine, thank you<br> | ||
'''''TurkerTunali''''': tbayen<br> | '''''TurkerTunali''''': tbayen<br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
'''''norbertbede''''': hi all<br> | '''''norbertbede''''': hi all<br> | ||
'''''norbertbede''''': Deepak i have reported this allready<br> | '''''norbertbede''''': Deepak i have reported this allready<br> | ||
'''''norbertbede''''': import inventory with costs<br> | '''''norbertbede''''': import inventory with costs<br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
'''''Deepak''''': Carlos, Due to IDEMPIERE-2057, it is not possible to import inventory with cost<br> | '''''Deepak''''': Carlos, Due to IDEMPIERE-2057, it is not possible to import inventory with cost<br> | ||
'''''Deepak''''': Is this known issue?<br> | '''''Deepak''''': Is this known issue?<br> | ||
| Line 112: | Line 77: | ||
'''''Deepak''''': I am wondering why there is below code in InventoryLine.BeforeSave<br> | '''''Deepak''''': I am wondering why there is below code in InventoryLine.BeforeSave<br> | ||
'''''Deepak''''': if (cost == null) { if (!MCostElement.COSTINGMETHOD_StandardCosting.equals(costingMethod)) { log.saveError("NoCostingRecord", ""); return false; } }<br> | '''''Deepak''''': if (cost == null) { if (!MCostElement.COSTINGMETHOD_StandardCosting.equals(costingMethod)) { log.saveError("NoCostingRecord", ""); return false; } }<br> | ||
| − | |||
| − | |||
| − | |||
'''''Deepak''''': Nobert thanks for offering help, I need to go to bed<br> | '''''Deepak''''': Nobert thanks for offering help, I need to go to bed<br> | ||
'''''Deepak''''': I have some confusion, and a fix, will check with Guru who wrote this piece to make sure that what I am doing is correct<br> | '''''Deepak''''': I have some confusion, and a fix, will check with Guru who wrote this piece to make sure that what I am doing is correct<br> | ||
'''''norbertbede''''': ok bye<br> | '''''norbertbede''''': ok bye<br> | ||
'''''norbertbede''''': CarlosRuiz i have tested ZK and found https://idempiere.atlassian.net/browse/IDEMPIERE-2099 issue cause other 3-4 issues reported by me<br> | '''''norbertbede''''': CarlosRuiz i have tested ZK and found https://idempiere.atlassian.net/browse/IDEMPIERE-2099 issue cause other 3-4 issues reported by me<br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
'''''nmicoud''''': Hi CarlosRuiz, any chance you can review https://idempiere.atlassian.net/browse/IDEMPIERE-2297 (which try to set the default language according to browser's settings) ?<br> | '''''nmicoud''''': Hi CarlosRuiz, any chance you can review https://idempiere.atlassian.net/browse/IDEMPIERE-2297 (which try to set the default language according to browser's settings) ?<br> | ||
'''''CarlosRuiz''''': sure nmicoud - let me try - I've been documenting -> http://wiki.idempiere.org/en/Category:New_Features_v2.1<br> | '''''CarlosRuiz''''': sure nmicoud - let me try - I've been documenting -> http://wiki.idempiere.org/en/Category:New_Features_v2.1<br> | ||
'''''nmicoud''''': ah great, thanks, will have a look, but after dinner, getting late here :)<br> | '''''nmicoud''''': ah great, thanks, will have a look, but after dinner, getting late here :)<br> | ||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
| − | |||
Latest revision as of 15:52, 12 November 2014
⇐ Table of Contents | Full Meeting Minutes | Full Meeting 2014-11-05
CarlosRuiz: Good Morning
nmicoud: Bonjour
CarlosRuiz: merging zk7 version
Deepak: Morning
Deepak: Are you merging zk7 to development?
aguerra: Buenos dias, good morning!!!!
CarlosRuiz: Hi Deepak - Hi Alejandro
CarlosRuiz: not yet Deepak - I'm merging dev into zk7 first and integrating fixes from hiep
Deepak: ohk
Deepak: I am testing import inventory with cost update having ASI level costing
Deepak: And found one bug and looks may have more on my way as this not looks tested
hieplq: hi Deepak i not sure i understand your requiremnet relate to question in prev meeting. "How can we create pull request for single commit?" maybe two branch as tbayen suggrest and graft command of hg can help
Deepak: Thanks hieplq, I yet need to try that
Deepak: I have some ticket to push in dev, So will do that
hieplq: it's easy to use TortoiseHg. i just know it when i bring some my local patch into zk7
TurkerTunali: goodmorning
TurkerTunali: it is 15:00 here
CarlosRuiz: Hi TurkerTunali
tbayen: Daarestiet
TurkerTunali: I've accidentally right clicked and found 3 new messages to translate
TurkerTunali: Value Preference
TurkerTunali: we are running out of time, I 've to find a client to get give support
hieplq: hi tbayen, i has a patch for IDEMPIERE-2132. you can test it and confirm? i apply mail structure of gmail and it's ok in my side.
tbayen: Thanks. I do not really understand what the difference is but I will review that.
tbayen: TurkerTunali, I can not understand your problem. Can you explain it again? Do you want some messages to be translated?
TurkerTunali: no no, I am fine, thank you
TurkerTunali: tbayen
norbertbede: hi all
norbertbede: Deepak i have reported this allready
norbertbede: import inventory with costs
Deepak: Carlos, Due to IDEMPIERE-2057, it is not possible to import inventory with cost
Deepak: Is this known issue?
CarlosRuiz: haven't checked - there are comments from norbertbede and hieplq in the ticket
Deepak: reading same
hieplq: i'm comeback you mean which ticket? maybe i can help @Deepak
Deepak: IDEMPIERE-2057
Deepak: I am not able to import inventory with cost
hieplq: sorry, i'm dummy in cost zone, just show what i know by read code in my comment :D
Deepak: hieplq/norbert, I am testing costing at batch/lot level and for first time inventory import
Deepak: And step on ticket did not worked
Deepak: I added code mentioned by Low and yet it did not worked
Deepak: I may need to investigate on fresh mind tomorrow,
hieplq: your test case is same norbert? you has a import file with muiltiple cost type
TurkerTunali: should Country list be in the reference?
nmicoud: no, there is a table
nmicoud: C_Country
TurkerTunali: can we change or set a default country?
nmicoud: AFAIR, it is based on the tenant language
TurkerTunali: we can't set TR so it is not an option
TurkerTunali: default coutry is United States on my instance
TurkerTunali: ups sorry, do you mean tenant language on the client setup?
TurkerTunali: let me check it
norbertbede: hi hieplq, deepak. my test case was org level costing not batch
Deepak: I am working to fix something and testing if that works
norbertbede: we made one time hotfix, not yet published
norbertbede: CarlosRuiz higlight me the follow ticket could make issue
norbertbede: https://idempiere.atlassian.net/browse/IDEMPIERE-22
norbertbede: Deepak i can help you test it well
Deepak: norbert, looks it works only with standard cost
Deepak: not with other then standard cost
Deepak: Is that true?
CarlosRuiz: :q
norbertbede: what i did
CarlosRuiz: :-) I'm writing vi commands on IRC
norbertbede: i import inventory with standard cost, becasue Average doesnt worked for us
norbertbede: cant find related ticket now :-S
Deepak: FYI, I found cause why it fails in case of Mandatory Attribute Set
Deepak: I fixed that
Deepak: I am wondering why there is below code in InventoryLine.BeforeSave
Deepak: if (cost == null) { if (!MCostElement.COSTINGMETHOD_StandardCosting.equals(costingMethod)) { log.saveError("NoCostingRecord", ""); return false; } }
Deepak: Nobert thanks for offering help, I need to go to bed
Deepak: I have some confusion, and a fix, will check with Guru who wrote this piece to make sure that what I am doing is correct
norbertbede: ok bye
norbertbede: CarlosRuiz i have tested ZK and found https://idempiere.atlassian.net/browse/IDEMPIERE-2099 issue cause other 3-4 issues reported by me
nmicoud: Hi CarlosRuiz, any chance you can review https://idempiere.atlassian.net/browse/IDEMPIERE-2297 (which try to set the default language according to browser's settings) ?
CarlosRuiz: sure nmicoud - let me try - I've been documenting -> http://wiki.idempiere.org/en/Category:New_Features_v2.1
nmicoud: ah great, thanks, will have a look, but after dinner, getting late here :)
