[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-21 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17162501#comment-17162501
 ] 

ASF subversion and git services commented on OFBIZ-11894:
-

Commit 679b2883e143105f35313807b4fc028b65c1e092 in ofbiz-framework's branch 
refs/heads/trunk from Priya Sharma
[ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=679b288 ]

Improved : Convert createRoleType service to entity-auto from Java and update 
its service calls (OFBIZ-11894) (#215)

(OFBIZ-11894)
* Improved: Convert createRoleType service to entity-auto from Java and update 
its service calls
- Removed the java implementation
- Converted the service to entity-auto
- updated the service call in Party tests where this service was used.
* Improved: Removed the testCreateRoleType test, as the base service is now 
converted to entity-auto and will be handled in entity-auto tests. (OFBIZ-11984)

Co-authored-by: Priya Sharma 

Thanks: Pawan Verma for review.

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Suraj Khurana
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-21 Thread ASF subversion and git services (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17162500#comment-17162500
 ] 

ASF subversion and git services commented on OFBIZ-11894:
-

Commit 679b2883e143105f35313807b4fc028b65c1e092 in ofbiz-framework's branch 
refs/heads/trunk from Priya Sharma
[ https://gitbox.apache.org/repos/asf?p=ofbiz-framework.git;h=679b288 ]

Improved : Convert createRoleType service to entity-auto from Java and update 
its service calls (OFBIZ-11894) (#215)

(OFBIZ-11894)
* Improved: Convert createRoleType service to entity-auto from Java and update 
its service calls
- Removed the java implementation
- Converted the service to entity-auto
- updated the service call in Party tests where this service was used.
* Improved: Removed the testCreateRoleType test, as the base service is now 
converted to entity-auto and will be handled in entity-auto tests. (OFBIZ-11984)

Co-authored-by: Priya Sharma 

Thanks: Pawan Verma for review.

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Suraj Khurana
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-20 Thread Suraj Khurana (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17161252#comment-17161252
 ] 

Suraj Khurana commented on OFBIZ-11894:
---

Looks good, let's wait for couple of days in case of any further suggestions or 
reviews.

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-20 Thread Pawan Verma (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17161013#comment-17161013
 ] 

Pawan Verma commented on OFBIZ-11894:
-

Thanks, [~priya.sharma.9362]

[~surajk] I've approved the PR, if it looks good to you please go ahead and 
merge :) 

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-20 Thread Priya Sharma (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17160998#comment-17160998
 ] 

Priya Sharma commented on OFBIZ-11894:
--

Thanks Suraj and Pawan!
I have updated the PR with the suggested change.

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-19 Thread Pawan Verma (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17160898#comment-17160898
 ] 

Pawan Verma commented on OFBIZ-11894:
-

+1 Suraj, Good point.

[~priya.sharma.9362] Lets remove the test case, it's not needed now.

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-19 Thread Suraj Khurana (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17160894#comment-17160894
 ] 

Suraj Khurana commented on OFBIZ-11894:
---

+1, looks good to me.

Also, shouldn't we remove the testCreateRoleType service as base service has 
been moved to entity-auto? We have entity-auto tests available already. WDYT?

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-19 Thread Pawan Verma (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17160667#comment-17160667
 ] 

Pawan Verma commented on OFBIZ-11894:
-

Hi [~priya.sharma.9362]

Thanks for your quick work, it looks good to me :)

I've tested it locally it works well, thanks again! will merge soon.

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-19 Thread Priya Sharma (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17160657#comment-17160657
 ] 

Priya Sharma commented on OFBIZ-11894:
--

Hi Pawan,
I have created a PR for this task.
Please review it and let me know if any updates are needed. TIA!

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Assignee: Priya Sharma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (OFBIZ-11894) Convert createRoleType service to entity-auto from Java and update its service calls

2020-07-17 Thread Suraj Khurana (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11894?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17159964#comment-17159964
 ] 

Suraj Khurana commented on OFBIZ-11894:
---

+1

> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> 
>
> Key: OFBIZ-11894
> URL: https://issues.apache.org/jira/browse/OFBIZ-11894
> Project: OFBiz
>  Issue Type: Improvement
>  Components: party
>Affects Versions: Trunk
>Reporter: Pawan Verma
>Priority: Minor
>
> Convert createRoleType service to entity-auto from Java and update its 
> service calls
> Service should return roleTypeId instead of GenericValue.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)