[Kmymoney-devel] Review Request 119911: Updated extraction code to support "Financial Times UK Funds" source for online price quotes

[Ian Neal]

Hi Allan,

Does it work with just:
http://funds.ft.com/uk/Tearsheet/Summary?s=%1

so not using either of:
http://funds.ft.com/uk/Tearsheet/Summary?s=%1:GBP
or
http://funds.ft.com/uk/Tearsheet/Summary?s=%1:GBX

Regards,

Ian

On 23/08/14 13:40, Allan wrote:
> On 23/08/14 13:15, Ian Neal wrote:
>> Hi Allan,
>>
>> Does it work any better with no symbol at the end, i.e.:
>>
>> http://funds.ft.com/uk/Tearsheet/Summary?s=%1
>>
>> At least one of my investments only works with configured that way.
>>
>> Ian
>
>
> Hi Neal
>
> It works with the ':GBX' suffix after the url, or after the symbol, 
> but not both.
>
> 'GBP' appears not to work at all.
>
> Allan
>
>
>> On 23/08/14 12:11, Allan wrote:
>>> On 23/08/14 12:05, David Houlden wrote:
>>>> Hi all,
>>>>
>>>> It was me that sent this change to Thomas. I just checked and I have
>>>> subsequently made another change to the URL. My version now has GBX
>>>> instead of GBP. If you can give me a while I will check Thomas's patch
>>>> to verify it works for me as I am using that source for prices. Bit 
>>>> busy
>>>> just now but will get back to you later.
>>>>
>>>> David.
>>>
>>>
>>> I've just created a new investment.  I've tried both the new and the
>>> old versions, and neither works for me.
>>>
>>> Allan
>>>
>>>> On Saturday 23 August 2014 10:52:36 Cristian Oneț wrote:
>>>>
>>>> This is an automatically generated e-mail. To reply, visit:
>>>> https://git.reviewboard.kde.org/r/119911/
>>>>
>>>>
>>>> On August 23rd, 2014, 10:43 a.m. UTC, Cristian Onețwrote:
>>>>
>>>> Ship It!
>>>>
>>>> On August 23rd, 2014, 10:44 a.m. UTC, Cristian Onețwrote:
>>>>
>>>> Although the currency symbol could be made configurable in the URL. 
>>>> For
>>>> fund in euro, like
>>>> http://funds.ft.com/uk/Tearsheet/Summary?s=ES0102562032:EUR
>>>>
>>>> On August 23rd, 2014, 10:49 a.m. UTC, Thomas Baumgartwrote:
>>>>
>>>> Right, but we only have the symbol of the security. This is something
>>>> the user must take care of, since we don't know which currency to 
>>>> use at
>>>> this point. Also, this is a completely different URL.
>>>>
>>>> OK, then ship it :).
>>>>
>>>>
>>>> - Cristian
>>>>
>>>> On August 23rd, 2014, 10:40 a.m. UTC, Thomas Baumgart wrote:
>>>>
>>>> Review request for KMymoney.
>>>>
>>>> By Thomas Baumgart.
>>>>
>>>> Updated Aug. 23, 2014, 10:40 a.m.
>>>>
>>>> Repository: kmymoney
>>>>
>>>> Description
>>>>
>>>> This patch shall support "Financial Times UK Funds" as source for 
>>>> online
>>>> price quotes again. Apparently their format changed. I received this
>>>> patch a long time ago via e-mail and just found it sitting in my 
>>>> stash.
>>>> Before I add it to master, I wanted someone to review it.
>>>>
>>>> Testing
>>>>
>>>> Compile, Link, Run the application but no testing towards the new 
>>>> source
>>>>
>>>> Diffs
>>>>
>>>>   * kmymoney/converter/webpricequote.cpp (7b1d24d)
>>>>
>>>> View Diff <https://git.reviewboard.kde.org/r/119911/diff/>
>>>>
>>>>
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> KMyMoney-devel mailing list
>>>> KMyMoney-devel at kde.org
>>>> https://mail.kde.org/mailman/listinfo/kmymoney-devel
>>>>
>>> _______________________________________________
>>> KMyMoney-devel mailing list
>>> KMyMoney-devel at kde.org
>>> https://mail.kde.org/mailman/listinfo/kmymoney-devel
>>