[ 
https://issues.apache.org/jira/browse/OFBIZ-1910?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12620577#action_12620577
 ] 

Bilgin Ibryam commented on OFBIZ-1910:
--------------------------------------

Hi Ashish,

I also wanted to add from - thru date fields to a few role entities, but the 
procedure for modifying a primary key field is very time consuming task.
If I got it right from 
http://docs.ofbiz.org/display/OFBTECH/General+Entity+Overview  , you have to 
create a new table(with the new primary key) and also provide a service to move 
the data.

+ 1 for having fromDate & thruDate fields in CustRequestRole entity. 
I think there are also other entities that need these very important fields: 
ProductStoreGroupRole, FacilityGroupRole, FacilityRole, AgreementRole, 
MarketingCampaignRole, SegmentGroupRole.

Regards,
Bilgin Ibryam

PS: here is the discussion: 
http://www.nabble.com/*Role-entities-question-to18503776.html#a18607087

> Add support of fromDate & thruDate in CustRequestRole entity
> ------------------------------------------------------------
>
>                 Key: OFBIZ-1910
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1910
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: order
>            Reporter: Ashish Vijaywargiya
>            Assignee: Ashish Vijaywargiya
>            Priority: Trivial
>         Attachments: Improve_CustRequestRole.patch
>
>
> Hello,
> I have added fromDate & thruDate field to maintain the old records from 
> CustRequestRole entity.
> Suppose a request is assigned to a party in the role of "Request Taker" and 
> after few day it can be assigned to another party in the role of "Request 
> Taker".
> So it will be easy to keep the old record with the help of fromDate & 
> thruDate.
> Please let me know your thoughts on it.
> --
> Ashish

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to