Re: [Dev] [AF Tools] use "master" as the label for default branch instead of "trunk"

2015-09-15 Thread Kavith Lokuhewage
Hi Jasintha,

Please find my comments inline.

On Tue, Sep 15, 2015 at 8:48 AM, Jasintha Dasanayake 
wrote:

> Hi Kavith
>
> Yes , we should not keep any mapping in DevS side , once this change is
> done  in AF this should be reflected into DevS so there shouldn't be
> anything to change in DevS side right ?
>
Yes, no changes required in DevS.

Thanks,
Kavith Lokuhewage


>
>
On Mon, Sep 14, 2015 at 12:39 PM, Manjula Rathnayake 
> wrote:
>
>> Hi Kavith,
>>
>> IMO, we should not store any mapping in DevStuido level. This needs to be
>> fixed in AF REST API level as well as in backends of AF and it require
>> considerable time to test and do any data migration if required too.
>> Since this is not a blocker, we will fix this in M6 milestone and you can
>> incorporate the changes to next release of Dev Studio or do a AF feature
>> release with other UX improvements.
>>
>> thank you.
>>
>>
>> On Fri, Sep 11, 2015 at 6:06 PM, Danushka Fernando 
>> wrote:
>>
>>> I guess we use trunk because we started this with svn and then moved on
>>> to git. +1 for renaming. @Manjula WDYT?
>>>
>>> Thanks & Regards
>>> Danushka Fernando
>>> Senior Software Engineer
>>> WSO2 inc. http://wso2.com/
>>> Mobile : +94716332729
>>>
>>> On Fri, Sep 11, 2015 at 5:00 PM, Kavith Lokuhewage 
>>> wrote:
>>>
 Hi,

 This is regarding $subject raised in [1] and [2].

 Currently the default version of an application is given the version
 "trunk" (which corresponds to "master" branch of the git repo). When
 checking out a version, DevStudio checks whether the user is checking out
 the version "trunk" and if so, sets "master" as the branch to checkout.
 However, when labeling the list of apps in tree view, we are not doing
 a mapping from "trunk" to "master". Rather we display the list of versions
 returned from AF API as it is.
 To fix this, either we have to maintain such a mapping in DevStudio
 side or this can be fixed from the AF side itself - by renaming version
 "trunk" to "master".
 Most importantly, it will be a bad experience for the user if we use
 two different version names in AF UI and DevS UI, to display this  default
 version.
 Is it possible (or make sense) to change this from AppFactory? Please
 share your thoughts.


 [1] https://wso2.org/jira/browse/TOOLS-3142
 [2] https://wso2.org/jira/browse/TOOLS-3143

 Thanks,

 --
 *Kavith Lokuhewage*
 Software Engineer
 WSO2 Inc. - http://wso2.com
 lean . enterprise . middleware
 Mobile - +9477-9-145-123 | +9471-455-6-401
 Linkedin 
 Twitter 

 ___
 Dev mailing list
 Dev@wso2.org
 http://wso2.org/cgi-bin/mailman/listinfo/dev


>>>
>>
>>
>> --
>> Manjula Rathnayaka
>> Associate Technical Lead
>> WSO2, Inc.
>> Mobile:+94 77 743 1987
>>
>> ___
>> Dev mailing list
>> Dev@wso2.org
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>
>>
>
>
> --
>
> *Jasintha Dasanayake*
>
> *Senior Software EngineerWSO2 Inc. | http://wso2.com
> lean . enterprise . middleware*
>
>
> *mobile :- 0711368118*
>



-- 
*Kavith Lokuhewage*
Software Engineer
WSO2 Inc. - http://wso2.com
lean . enterprise . middleware
Mobile - +9477-9-145-123 | +9471-455-6-401
Linkedin   Twitter

___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [AF Tools] use "master" as the label for default branch instead of "trunk"

2015-09-14 Thread Jasintha Dasanayake
Hi Kavith

Yes , we should not keep any mapping in DevS side , once this change is
done  in AF this should be reflected into DevS so there shouldn't be
anything to change in DevS side right ?

Thanks and Regards
/Jasintha


On Mon, Sep 14, 2015 at 12:39 PM, Manjula Rathnayake 
wrote:

> Hi Kavith,
>
> IMO, we should not store any mapping in DevStuido level. This needs to be
> fixed in AF REST API level as well as in backends of AF and it require
> considerable time to test and do any data migration if required too.
> Since this is not a blocker, we will fix this in M6 milestone and you can
> incorporate the changes to next release of Dev Studio or do a AF feature
> release with other UX improvements.
>
> thank you.
>
>
> On Fri, Sep 11, 2015 at 6:06 PM, Danushka Fernando 
> wrote:
>
>> I guess we use trunk because we started this with svn and then moved on
>> to git. +1 for renaming. @Manjula WDYT?
>>
>> Thanks & Regards
>> Danushka Fernando
>> Senior Software Engineer
>> WSO2 inc. http://wso2.com/
>> Mobile : +94716332729
>>
>> On Fri, Sep 11, 2015 at 5:00 PM, Kavith Lokuhewage 
>> wrote:
>>
>>> Hi,
>>>
>>> This is regarding $subject raised in [1] and [2].
>>>
>>> Currently the default version of an application is given the version
>>> "trunk" (which corresponds to "master" branch of the git repo). When
>>> checking out a version, DevStudio checks whether the user is checking out
>>> the version "trunk" and if so, sets "master" as the branch to checkout.
>>> However, when labeling the list of apps in tree view, we are not doing a
>>> mapping from "trunk" to "master". Rather we display the list of versions
>>> returned from AF API as it is.
>>> To fix this, either we have to maintain such a mapping in DevStudio side
>>> or this can be fixed from the AF side itself - by renaming version "trunk"
>>> to "master".
>>> Most importantly, it will be a bad experience for the user if we use two
>>> different version names in AF UI and DevS UI, to display this  default
>>> version.
>>> Is it possible (or make sense) to change this from AppFactory? Please
>>> share your thoughts.
>>>
>>>
>>> [1] https://wso2.org/jira/browse/TOOLS-3142
>>> [2] https://wso2.org/jira/browse/TOOLS-3143
>>>
>>> Thanks,
>>>
>>> --
>>> *Kavith Lokuhewage*
>>> Software Engineer
>>> WSO2 Inc. - http://wso2.com
>>> lean . enterprise . middleware
>>> Mobile - +9477-9-145-123 | +9471-455-6-401
>>> Linkedin 
>>> Twitter 
>>>
>>> ___
>>> Dev mailing list
>>> Dev@wso2.org
>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>
>>>
>>
>
>
> --
> Manjula Rathnayaka
> Associate Technical Lead
> WSO2, Inc.
> Mobile:+94 77 743 1987
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>


-- 

*Jasintha Dasanayake*

*Senior Software EngineerWSO2 Inc. | http://wso2.com lean
. enterprise . middleware*


*mobile :- 0711368118*
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [AF Tools] use "master" as the label for default branch instead of "trunk"

2015-09-14 Thread Manjula Rathnayake
Hi Kavith,

IMO, we should not store any mapping in DevStuido level. This needs to be
fixed in AF REST API level as well as in backends of AF and it require
considerable time to test and do any data migration if required too.
Since this is not a blocker, we will fix this in M6 milestone and you can
incorporate the changes to next release of Dev Studio or do a AF feature
release with other UX improvements.

thank you.


On Fri, Sep 11, 2015 at 6:06 PM, Danushka Fernando 
wrote:

> I guess we use trunk because we started this with svn and then moved on to
> git. +1 for renaming. @Manjula WDYT?
>
> Thanks & Regards
> Danushka Fernando
> Senior Software Engineer
> WSO2 inc. http://wso2.com/
> Mobile : +94716332729
>
> On Fri, Sep 11, 2015 at 5:00 PM, Kavith Lokuhewage 
> wrote:
>
>> Hi,
>>
>> This is regarding $subject raised in [1] and [2].
>>
>> Currently the default version of an application is given the version
>> "trunk" (which corresponds to "master" branch of the git repo). When
>> checking out a version, DevStudio checks whether the user is checking out
>> the version "trunk" and if so, sets "master" as the branch to checkout.
>> However, when labeling the list of apps in tree view, we are not doing a
>> mapping from "trunk" to "master". Rather we display the list of versions
>> returned from AF API as it is.
>> To fix this, either we have to maintain such a mapping in DevStudio side
>> or this can be fixed from the AF side itself - by renaming version "trunk"
>> to "master".
>> Most importantly, it will be a bad experience for the user if we use two
>> different version names in AF UI and DevS UI, to display this  default
>> version.
>> Is it possible (or make sense) to change this from AppFactory? Please
>> share your thoughts.
>>
>>
>> [1] https://wso2.org/jira/browse/TOOLS-3142
>> [2] https://wso2.org/jira/browse/TOOLS-3143
>>
>> Thanks,
>>
>> --
>> *Kavith Lokuhewage*
>> Software Engineer
>> WSO2 Inc. - http://wso2.com
>> lean . enterprise . middleware
>> Mobile - +9477-9-145-123 | +9471-455-6-401
>> Linkedin 
>> Twitter 
>>
>> ___
>> Dev mailing list
>> Dev@wso2.org
>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>
>>
>


-- 
Manjula Rathnayaka
Associate Technical Lead
WSO2, Inc.
Mobile:+94 77 743 1987
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


[Dev] [AF Tools] use "master" as the label for default branch instead of "trunk"

2015-09-11 Thread Kavith Lokuhewage
Hi,

This is regarding $subject raised in [1] and [2].

Currently the default version of an application is given the version
"trunk" (which corresponds to "master" branch of the git repo). When
checking out a version, DevStudio checks whether the user is checking out
the version "trunk" and if so, sets "master" as the branch to checkout.
However, when labeling the list of apps in tree view, we are not doing a
mapping from "trunk" to "master". Rather we display the list of versions
returned from AF API as it is.
To fix this, either we have to maintain such a mapping in DevStudio side or
this can be fixed from the AF side itself - by renaming version "trunk" to
"master".
Most importantly, it will be a bad experience for the user if we use two
different version names in AF UI and DevS UI, to display this  default
version.
Is it possible (or make sense) to change this from AppFactory? Please share
your thoughts.


[1] https://wso2.org/jira/browse/TOOLS-3142
[2] https://wso2.org/jira/browse/TOOLS-3143

Thanks,

-- 
*Kavith Lokuhewage*
Software Engineer
WSO2 Inc. - http://wso2.com
lean . enterprise . middleware
Mobile - +9477-9-145-123 | +9471-455-6-401
Linkedin   Twitter

___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev


Re: [Dev] [AF Tools] use "master" as the label for default branch instead of "trunk"

2015-09-11 Thread Danushka Fernando
I guess we use trunk because we started this with svn and then moved on to
git. +1 for renaming. @Manjula WDYT?

Thanks & Regards
Danushka Fernando
Senior Software Engineer
WSO2 inc. http://wso2.com/
Mobile : +94716332729

On Fri, Sep 11, 2015 at 5:00 PM, Kavith Lokuhewage  wrote:

> Hi,
>
> This is regarding $subject raised in [1] and [2].
>
> Currently the default version of an application is given the version
> "trunk" (which corresponds to "master" branch of the git repo). When
> checking out a version, DevStudio checks whether the user is checking out
> the version "trunk" and if so, sets "master" as the branch to checkout.
> However, when labeling the list of apps in tree view, we are not doing a
> mapping from "trunk" to "master". Rather we display the list of versions
> returned from AF API as it is.
> To fix this, either we have to maintain such a mapping in DevStudio side
> or this can be fixed from the AF side itself - by renaming version "trunk"
> to "master".
> Most importantly, it will be a bad experience for the user if we use two
> different version names in AF UI and DevS UI, to display this  default
> version.
> Is it possible (or make sense) to change this from AppFactory? Please
> share your thoughts.
>
>
> [1] https://wso2.org/jira/browse/TOOLS-3142
> [2] https://wso2.org/jira/browse/TOOLS-3143
>
> Thanks,
>
> --
> *Kavith Lokuhewage*
> Software Engineer
> WSO2 Inc. - http://wso2.com
> lean . enterprise . middleware
> Mobile - +9477-9-145-123 | +9471-455-6-401
> Linkedin 
> Twitter 
>
> ___
> Dev mailing list
> Dev@wso2.org
> http://wso2.org/cgi-bin/mailman/listinfo/dev
>
>
___
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev