girishvasmatkar opened a new pull request #214: URL: https://github.com/apache/ofbiz-framework/pull/214 Implemented: New service definition attribute "verb" to denote the corresponding HTTP method (OFBIZ-11328) Explanation In order for exportable services in OFBiz to be called via REST interface, it is necessary for the service to tell what HTTP verb should be used to call it. This implementation adds the attribute and makes it available in ModelService class via property **verb**. Thanks: Girish ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
In reply to this post by GitBox
girishvasmatkar commented on pull request #214: URL: https://github.com/apache/ofbiz-framework/pull/214#issuecomment-657495968 I'm keeping it open for review by the reviewers. I will merge it during the upcoming weekend. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
In reply to this post by GitBox
PierreSmits commented on pull request #214: URL: https://github.com/apache/ofbiz-framework/pull/214#issuecomment-659888727 If we're talking about an action here, why don't we call it 'action'? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
In reply to this post by GitBox
girishvasmatkar commented on pull request #214: URL: https://github.com/apache/ofbiz-framework/pull/214#issuecomment-659904734 @PierreSmits : "action" makes more sense. I'll change the attribute name and then merge. Thanks. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
In reply to this post by GitBox
girishvasmatkar merged pull request #214: URL: https://github.com/apache/ofbiz-framework/pull/214 ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [hidden email] |
Free forum by Nabble | Edit this page |