Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Jan 30, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

EvanHahn and others added 2 commits January 29, 2026 16:44
*This test-only change should have no user impact.*
)

ref
https://github.com/TryGhost/Ghost/actions/runs/21494158108/job/61924761703

One of the newsletter tests is [flaky in CI][0]. I think this is because
app settings are loading and we redirect away before they do.

The old logic was, roughly: redirect if newsletters are disabled *or* if
app settings are still loading.

The new logic is: redirect if newsletters are disabled. Don't redirect
if app settings are still loading.

I'm not certain this will fix CI flakiness--I can't reproduce
locally--but I think this is an improvement anyway.

[0]:
https://github.com/TryGhost/Ghost/actions/runs/21494158108/job/61924761703

---------

Co-authored-by: Chris Raible <chris@ghost.org>
@pull pull bot locked and limited conversation to collaborators Jan 30, 2026
@pull pull bot added the ⤵️ pull label Jan 30, 2026
@pull pull bot merged commit e3e2d6c into code:main Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant