search

thisisparker / xword-dl

⬛⬜⬛ Command line tool to scrape crosswords from online solvers and save them as .puz files ⬛⬜⬛
MIT License
145 stars 31 forks source link

Fix new yorker download with new selector #110

Closed thisisparker closed 1 year ago

thisisparker commented 1 year ago

Fixes #109, but could use a little more testing.

edsantiago commented 1 year ago

LGTM. Works for tny 2023-06-{28,29,30}, at least superficially (I have not solved any yet). I will report back later today with a yea/nay. Thank you!

thisisparker commented 1 year ago

I can also report that it works for some spot-checked older puzzles that predate this formatting change.

edsantiago commented 1 year ago

I'm semi-sure that the formatting change is global, i.e. a change in the page generation code (vs static pages). That is: yesterday I tried xword-dl on an older page, and it failed the same way. So it's expected that your new code will work on past pages.

I just did Wednesday and Thursday, happy-pencil on both, suggesting that everything is working fine. Once again, LGTM.