Dominik Stadler

Results 77 comments of Dominik Stadler

Thanks for the changes. Looks good on a brief look. Can you add a sample .msi-file to directory test-data/poi-fs and a small unit-test which verifies opening it so we verify...

The duplication seems to be because of the "AlternatContent" (i.e. MSs way of not changing the spec, but still adding new types of content ...). Can you try to add...

See also https://stackoverflow.com/questions/68074154/are-there-mcalternatecontent-and-mcfallback-support-in-poi-xwpf-parser and https://stackoverflow.com/questions/46802369/replace-text-in-text-box-of-docx-by-using-apache-poi/46894499#46894499 for related discussions about "AlternateContent"

Ok, as said I am not sure if this will even make it work, but two things I see which might be off: * It's important to do the "workaround()"...

Maybe your version of module `urllib.request` is outdated? Which version do you have installed? On Debian/Ubuntu it is provided by package `python3-urllib3` The README does currently not state any minimum...

When I tried to load https://query1.finance.yahoo.com/v7/finance/quote?symbols=ORCL in the Browser, it mostly works, but sometimes I get ``` {"finance":{"result":null,"error":{"code":"Unauthorized","description":"Invalid Crumb"}}} ``` So it seems this may be some problem at Yahoo...

> use https://query1.finance.yahoo.com/v6/finance/quote?symbols=... > > it works for me....v7 -> v6 This means the following may be a workaround in the client-code without changes to the yahoofinance-api code itself ```...

It seems the "v6" API was now shutdown by Yahoo, likely because everyone switched over from "v7"

Thanks for the proposed changes. I will merge at least some of it when I find some time to take a closer look!

There are no plans to make this more "official", thus it likely will stay in this "demo" mode. Unfortunately there is not much time and community to build this out...