On Tue, Nov 18, 2014 at 5:20 PM, Tanya Madurapperuma <[email protected]> wrote:

> Verified in 1.6 and 1.7.
>
Sorry I mean 1.8 not 1.6
But In the code base of 1.6 also this api is there. Think we need to add in
1.6 docs as well.

Thanks,
Tanya

>
> Thanks,
> Tanya
>
> On Tue, Nov 18, 2014 at 5:14 PM, Nirdesha Munasinghe <[email protected]>
> wrote:
>
>> Hi Tanya,
>>
>> I fixed both 1.7.0 and 1.8.0 with the following info:
>>
>>    - To add a subscription by application ID: action=*addSubscription*
>>    &name=xxx&version=xxx&provider=xxx&tier=xxx&*applicationId*=xxx
>>    - To add a subscription by application name: action=
>>    *addAPISubscription*&name=xxx&version=xxx&provider=xxx&tier=xxx&
>>    *applicationName*=xxx
>>
>> Will be great if you can verify if this is correct.
>>
>> Thanks,
>> -Nirdesha
>>
>> On Tue, Nov 18, 2014 at 9:30 AM, Jackie Wheeler <[email protected]> wrote:
>>
>>> Nirdesha, can you take a look?
>>>
>>> Thanks,
>>> Jackie
>>>
>>> On Mon, Nov 17, 2014 at 1:17 AM, Roshan Wijesena <[email protected]>
>>> wrote:
>>>
>>>> Thanks Tanya/Sanjeewa.
>>>>
>>>> It would be great if we can get documented this somewhere in our docs?
>>>>
>>>> Regards
>>>> Roshan.
>>>>
>>>> On Mon, Nov 17, 2014 at 2:34 PM, Tanya Madurapperuma <[email protected]>
>>>> wrote:
>>>>
>>>>> subscription-add.jag [1] has
>>>>> else if (action == "addAPISubscription" && request.getMethod() ==
>>>>> 'POST')
>>>>> which takes application name to make a subscription. It calls the
>>>>> addAPISubscription method at [2] which calls the 
>>>>> jsFunction_addAPISubscription
>>>>> method of api store hostobject [3]
>>>>>
>>>>> Hence no need to write a new api.
>>>>>
>>>>> [1]
>>>>> https://github.com/wso2-dev/carbon-apimgt/blob/master/components/apimgt/api-store-web/src/site/blocks/subscription/subscription-add/ajax/subscription-add.jag
>>>>> [2]
>>>>> https://github.com/wso2-dev/carbon-apimgt/blob/master/components/apimgt/api-store-web/src/modules/subscription/add.jag
>>>>> [3]
>>>>> https://github.com/wso2-dev/carbon-apimgt/blob/master/components/apimgt/org.wso2.carbon.apimgt.hostobjects/src/main/java/org/wso2/carbon/apimgt/hostobjects/APIStoreHostObject.java
>>>>>
>>>>> Thanks,
>>>>> Tanya
>>>>>
>>>>> On Mon, Nov 17, 2014 at 2:17 PM, Sanjeewa Malalgoda <[email protected]
>>>>> > wrote:
>>>>>
>>>>>> It seems you need to expose API to pass *Application name* and user
>>>>>> name here. Previously mentioned action require *application ID*.
>>>>>> So we may be able to keep current API as it is and add new action as
>>>>>> you mentioned. So users who use existing rest API do not have
>>>>>> to change their application code.
>>>>>>
>>>>>> Thanks,
>>>>>> sanjeewa.
>>>>>>
>>>>>> On Mon, Nov 17, 2014 at 1:54 PM, Sanjeewa Malalgoda <
>>>>>> [email protected]> wrote:
>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> On Mon, Nov 17, 2014 at 1:36 PM, Tanya Madurapperuma <[email protected]
>>>>>>> > wrote:
>>>>>>>
>>>>>>>> Isn''t the *addAPISubscription* action in subscription-add.jag
>>>>>>>> does the same?
>>>>>>>>
>>>>>>> +1. Yes we may use same
>>>>>>>
>>>>>>> Thanks,
>>>>>>> sanjeewa.
>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Tanya
>>>>>>>>
>>>>>>>> On Mon, Nov 17, 2014 at 12:32 PM, Roshan Wijesena <[email protected]>
>>>>>>>> wrote:
>>>>>>>>
>>>>>>>>> Hi Devs,
>>>>>>>>>
>>>>>>>>> Current existing addSubscription API [1] accept Application Id as
>>>>>>>>> a parameter in POST request. It seems, use an internal  attribute as 
>>>>>>>>> an API
>>>>>>>>> parameter would not be a good approach.
>>>>>>>>>
>>>>>>>>> Further, I would like to propose a new API, which will take
>>>>>>>>> application name and user name  as  parameters instead of application 
>>>>>>>>> ID.
>>>>>>>>> Sample curl request for the proposed API will be as below.
>>>>>>>>>
>>>>>>>>> curl -X POST -b cookies
>>>>>>>>> http://localhost:9763/store/site/blocks/subscription/subscription-add/ajax/subscription-add.jag
>>>>>>>>>  -d
>>>>>>>>> 'action=addSubscriptionByName&name=API1&version=v1&provider=admin&tier=Gold&applicationName=PizzaShop&userName=admin'
>>>>>>>>>
>>>>>>>>> WDYT?
>>>>>>>>>
>>>>>>>>> [1] https://docs.wso2.com/display/AM170/Store+APIs.
>>>>>>>>>
>>>>>>>>> Regards
>>>>>>>>> Roshan.
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Roshan Wijesena.
>>>>>>>>> Senior Software Engineer-WSO2 Inc.
>>>>>>>>> Mobile: *+94752126789*
>>>>>>>>> Email: [email protected]
>>>>>>>>> *WSO2, Inc. :** wso2.com <http://wso2.com/>*
>>>>>>>>> lean.enterprise.middleware.
>>>>>>>>>
>>>>>>>>> _______________________________________________
>>>>>>>>> Dev mailing list
>>>>>>>>> [email protected]
>>>>>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> Tanya Madurapperuma
>>>>>>>>
>>>>>>>> Software Engineer,
>>>>>>>> WSO2 Inc. : wso2.com
>>>>>>>> Mobile : +94718184439
>>>>>>>> Blog : http://tanyamadurapperuma.blogspot.com
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> Dev mailing list
>>>>>>>> [email protected]
>>>>>>>> http://wso2.org/cgi-bin/mailman/listinfo/dev
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>>
>>>>>>> *Sanjeewa Malalgoda*
>>>>>>> WSO2 Inc.
>>>>>>> Mobile : +94713068779
>>>>>>>
>>>>>>>  <http://sanjeewamalalgoda.blogspot.com/>blog
>>>>>>> :http://sanjeewamalalgoda.blogspot.com/
>>>>>>> <http://sanjeewamalalgoda.blogspot.com/>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>>
>>>>>> *Sanjeewa Malalgoda*
>>>>>> WSO2 Inc.
>>>>>> Mobile : +94713068779
>>>>>>
>>>>>>  <http://sanjeewamalalgoda.blogspot.com/>blog
>>>>>> :http://sanjeewamalalgoda.blogspot.com/
>>>>>> <http://sanjeewamalalgoda.blogspot.com/>
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Tanya Madurapperuma
>>>>>
>>>>> Software Engineer,
>>>>> WSO2 Inc. : wso2.com
>>>>> Mobile : +94718184439
>>>>> Blog : http://tanyamadurapperuma.blogspot.com
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Roshan Wijesena.
>>>> Senior Software Engineer-WSO2 Inc.
>>>> Mobile: *+94752126789*
>>>> Email: [email protected]
>>>> *WSO2, Inc. :** wso2.com <http://wso2.com/>*
>>>> lean.enterprise.middleware.
>>>>
>>>> --
>>>> You received this message because you are subscribed to the Google
>>>> Groups "WSO2 Documentation" group.
>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>> an email to [email protected].
>>>> For more options, visit https://groups.google.com/a/wso2.com/d/optout.
>>>>
>>>
>>>
>>>
>>> --
>>> *Jackie Wheeler*
>>> VP, Technical Content
>>> WSO2, Inc.
>>> Mobile: +1 510 725-2876
>>> http://wso2.com/
>>>
>>
>>
>>
>> --
>>
>> Thanks,
>>
>> Nirdesha Munasinghe,
>> WSO2 Inc.
>> Web:http://wso2.com
>>
>> Mobile: +94 776321920
>>
>
>
>
> --
> Tanya Madurapperuma
>
> Software Engineer,
> WSO2 Inc. : wso2.com
> Mobile : +94718184439
> Blog : http://tanyamadurapperuma.blogspot.com
>



-- 
Tanya Madurapperuma

Software Engineer,
WSO2 Inc. : wso2.com
Mobile : +94718184439
Blog : http://tanyamadurapperuma.blogspot.com
_______________________________________________
Dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to