Q726kbXuN / nytxw_puz

Turn NY Times crosswords into Across Lite files
The Unlicense
41 stars 10 forks source link

Http clickthrough prevents puz gereration #19

Closed deliciouslysmug closed 2 years ago

deliciouslysmug commented 2 years ago

The http version of the crossword now serves up a popup with class="xwdmodal--body xwdstart-modal" and a button to proceed with class="pz-moment__button". I believe those have to be acted upon before being able to access the puzzle information.

At the current moment, it appears that .puz generation of NYT is broken.

Q726kbXuN commented 2 years ago

@deliciouslysmug Which version of nytxw_puz are you using?

Q726kbXuN commented 2 years ago

image

The latest version looks good to me, can you verify if there are still issues running 1.06 ?

deliciouslysmug commented 2 years ago

So sorry! Missed the 1.06 release. Closing this issue! Great job :)

buszidog commented 2 years ago

Wah! Version 1.06 didn't work on today's (Sunday) puzzle. It worked on this Friday's and Saturday's, last week's Sunday and the Sunday before. Agh, what, is the NYT planting bombs in it?

"I can't remember being a baby and I put that down to it being boring." -Karl Pilkington

On Thursday, March 24, 2022, 10:23:16 PM EDT, Q726kbXuN ***@***.***> wrote:  

The latest version looks good to me, can you verify if there are still issues running 1.06 ?

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this thread.Message ID: @.***>

edsantiago commented 2 years ago

@buszidog just git-fetch. Last night's c40e88e commit (untagged) fixes the issues with today's NYT.

buszidog commented 2 years ago

GitHub/code dummy here(geez, I'm gonna be 75, have mercy!), how do I include/amend/drop this into nyt_puz v. 1.06?Thank you, thank you!

Loraine

"People moan about drugs being tested on animals. I sort of think it depends innit. If the drug's aspirin and the monkey's got a headache, is it right?" -Karl Pilkington

On Sunday, March 27, 2022, 05:58:26 PM EDT, Ed Santiago ***@***.***> wrote:  

@buszidog just git-fetch. Last night's c40e88e commit (untagged) fixes the issues with today's NYT.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

Q726kbXuN commented 2 years ago

@edsantiago That's hilarious. The fix you're referencing was meant for an unrelated upcoming feature. For once I'm ahead of the game.

@buszidog Try downloading the pre-release of 1.07 from here. Let us know how it goes.

buszidog commented 2 years ago

It worked! Again, you're a genius. Thank you, thank you!