Fixed XETRA, Tradegate and SInvestor after webpage was restructured #353
Conversation
|
Awesome! Thanks for the quick response. BTW - Prefer most recent changes at the top of the list in |
bpschuck
added
modified scraping
|
changes reordered... |
|
@bpschuck Is something missing? |
No sir. I've just been extremely busy at the day job and personal life. Besides holiday related goodies, getting my late father's house prepped for sale (including repairing hurricane damage). |
|
Oh sry then please excuse my impatience. |
|
Not a problem. |
|
@mumpitzstuff - Before merging the pull request I executed the test suites for the 3 modules. t/extra.t and t/tradegate.t failed with errors. t/sinvestor.t successfully completed all 94 planned tests. I did not investigate further, but usually failures occur due to symbols no longer being valid. If you could please review and fix the test suites that would be greatly appreciated. |
- testcases fixed (used bond does not exist anymore)
|
The bond used for the tradegate and xetra tests was not valid anymore. Therefore i had to use another one (tests fixed). But now an error occured because bonds seems to have a different output. Therefore i adjusted sinvestor, xetra and tradegate another time to fix this issue hopefully. |
|
PR merged. Thank you. |
Fixes #350.