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

[Allan]

On 23/08/14 13:47, Ian Neal wrote:
> 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

Hi Neal

Yes, but...

The symbol has to match.  I did a look-up on the FT site and it showed 
the suffix :GBX.  So, I added that to the symbol.  You'll need to check 
yours.

Allan



> 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
>>>
>