PennyDreadfulMTG / Penny-Dreadful-Tools

A suite of tools for the Penny Dreadful MTGO community
https://pennydreadfulmagic.com
MIT License
41 stars 28 forks source link

Revert "Don't clear decksite rotation cache via web, be more direct" #13340

Closed bakert closed 1 week ago

bakert commented 1 week ago

This reverts commit d9dc4ab42f30fdc0eb93fa87e437878dc40af8e2.

While this is better it does not work on prod the way it's currently set up to run in a directory that is not a Penny-Dreadful-Tools checkout :(

silasary commented 4 days ago

For what it's worth, I don't want rotation script pulling in extra dependencies if we can avoid it.

On Tue, 19 Nov 2024, 4:56 am Thomas David Baker, @.***> wrote:

Merged #13340 https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340 into master.

— Reply to this email directly, view it on GitHub https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340#event-15338144706, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABPNTS3FG56CVDHXA3QJLL2BIS4HAVCNFSM6AAAAABSAHC7HKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGMZTQMJUGQ3TANQ . You are receiving this because you are subscribed to this thread.Message ID: <PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340/issue_event/15338144706@ github.com>

bakert commented 4 days ago

Yeah I agree. I was getting desperate! I found a way to make it work poking the web though by spawning a process that calls a maintenance task instead of running in a thread.

On Mon, Nov 25, 2024 at 1:58 AM Katelyn Gigante @.***> wrote:

For what it's worth, I don't want rotation script pulling in extra dependencies if we can avoid it.

On Tue, 19 Nov 2024, 4:56 am Thomas David Baker, @.***> wrote:

Merged #13340 https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340 into master.

— Reply to this email directly, view it on GitHub < https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340#event-15338144706>,

or unsubscribe < https://github.com/notifications/unsubscribe-auth/AABPNTS3FG56CVDHXA3QJLL2BIS4HAVCNFSM6AAAAABSAHC7HKVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGMZTQMJUGQ3TANQ>

. You are receiving this because you are subscribed to this thread.Message ID:

<PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340/issue_event/15338144706@ github.com>

— Reply to this email directly, view it on GitHub https://github.com/PennyDreadfulMTG/Penny-Dreadful-Tools/pull/13340#issuecomment-2497480196, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAAMBL24ECYFW252C4DSEL2CLYE5AVCNFSM6AAAAABSAHC7HKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJXGQ4DAMJZGY . You are receiving this because you modified the open/close state.Message ID: @.*** com>