On Thu, Apr 30, 2015 at 2:35 PM, Prabath Abeysekera <[email protected]>
wrote:

> Hi Milan,
>
> On Thu, Apr 30, 2015 at 2:13 PM, Milan Perera <[email protected]> wrote:
>
>> Hi Dulitha,
>>
>> I have added features to the relevant table. Please find the PR[1].
>>
>
> The way this's already been done doesn't seem to be right. IMO, persisting
> the feature metadata corresponding to a particular device platform needs to
> be done at the point where the plugin gets registered into the device
> management framework. The current approach hides some part of business
> logic (registering device specific feature metadata) within the database
> layer as the same is done through a SQL script, which should ideally have
> been captured in the plug-in implementation itself. Please review this
> again and try to get things improved along the approach suggested above.
>
​Are you saying to have another script to register features separately for
each plugin?

Also Dilshan/Milan can you please update this sheet [1] with the Operation
Constants so that we can make it similar as much as possible.

[1] -
https://docs.google.com/a/wso2.com/spreadsheets/d/1MSKfzHigj6JBgW5EiH4O08A7s4owl43S9oUlx-T8Gvo/edit?usp=sharing
​

>
> Cheers,
> Prabath
>
>
>>
>> [1] https://github.com/wso2/carbon-device-mgt-plugins/pull/4
>>
>> Regards,
>>
>> On Thu, Apr 30, 2015 at 9:51 AM, Milan Perera <[email protected]> wrote:
>>
>>> Hi Dulitha,
>>>
>>> Features have not been added yet. Will do ASAP.
>>>
>>> Regards,
>>>
>>> On Thu, Apr 30, 2015 at 7:51 AM, Dulitha Wijewantha <[email protected]>
>>> wrote:
>>>
>>>> Hi Milan,
>>>> Have the features been added to the database? (Last time I checked,
>>>> features weren't available in the database).
>>>> Cheers~
>>>>
>>>> On Thu, Apr 30, 2015 at 3:12 AM, Prabath Abeysekera <[email protected]>
>>>> wrote:
>>>>
>>>>> Milan, please share this via dev@.
>>>>>
>>>>> On Thu, Apr 30, 2015 at 1:12 AM, Milan Perera <[email protected]> wrote:
>>>>>
>>>>>> Hi all,
>>>>>>
>>>>>> $subject can be found here:
>>>>>> https://docs.google.com/a/wso2.com/document/d/1D5tcSh-uAT28e2nDLUzd4TQ-ohH27onekgS0E73CVHg/edit?usp=sharing
>>>>>>
>>>>>> Thanks,
>>>>>>
>>>>>> --
>>>>>> Milan Harindu Perera
>>>>>> Software Engineer
>>>>>> *WSO2, Inc*
>>>>>> (+94) 77 309 7088
>>>>>> lean . enterprise . middleware
>>>>>> <http://lk.linkedin.com/in/milanharinduperera>
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Prabath Abeysekara
>>>>> Technical Lead
>>>>> WSO2 Inc.
>>>>> Email: [email protected]
>>>>> Mobile: +94774171471
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Dulitha Wijewantha (Chan)
>>>> Software Engineer - Mobile Development
>>>> WSO2 Inc
>>>> Lean.Enterprise.Middleware
>>>>  * ~Email       [email protected] <[email protected]>*
>>>> *  ~Mobile     +94712112165 <%2B94712112165>*
>>>> *  ~Website   dulitha.me <http://dulitha.me>*
>>>> *  ~Twitter     @dulitharw <https://twitter.com/dulitharw>*
>>>>   *~Github     @dulichan <https://github.com/dulichan>*
>>>>   *~SO     @chan <http://stackoverflow.com/users/813471/chan>*
>>>>
>>>
>>>
>>>
>>> --
>>> Milan Harindu Perera
>>> Software Engineer
>>> *WSO2, Inc*
>>> (+94) 77 309 7088
>>> lean . enterprise . middleware
>>> <http://lk.linkedin.com/in/milanharinduperera>
>>>
>>
>>
>>
>> --
>> Milan Harindu Perera
>> Software Engineer
>> *WSO2, Inc*
>> (+94) 77 309 7088
>> lean . enterprise . middleware
>> <http://lk.linkedin.com/in/milanharinduperera>
>>
>
>
>
> --
> Prabath Abeysekara
> Technical Lead
> WSO2 Inc.
> Email: [email protected]
> Mobile: +94774171471
>



-- 
Dulitha Wijewantha (Chan)
Software Engineer - Mobile Development
WSO2 Inc
Lean.Enterprise.Middleware
 * ~Email       [email protected] <[email protected]>*
*  ~Mobile     +94712112165*
*  ~Website   dulitha.me <http://dulitha.me>*
*  ~Twitter     @dulitharw <https://twitter.com/dulitharw>*
  *~Github     @dulichan <https://github.com/dulichan>*
  *~SO     @chan <http://stackoverflow.com/users/813471/chan>*
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to