Skip to content

Remove final require_login() - #1685

Merged
cpeel merged 1 commit into
DistributedProofreaders:pgdp-productionfrom
cpeel:remove-final-require-login
Oct 26, 2026
Merged

cpeel merged 1 commit into
DistributedProofreaders:pgdp-productionfrom
cpeel:remove-final-require-login

Conversation

@cpeel

@cpeel cpeel commented Oct 14, 2026

Copy link
Copy Markdown
Member

There are 3 instances of require_login() in pgdp-production -- remove those.

@cpeel
cpeel requested a review from srjfoo October 14, 2026 16:19
@cpeel

cpeel commented Oct 14, 2026

Copy link
Copy Markdown
Member Author

I hotfixed these on PROD so they don't trigger a deprecation warning in php_errors.

@cpeel
cpeel force-pushed the remove-final-require-login branch from b10f8aa to 0c01ef7 Compare October 16, 2026 17:17
@cpeel
cpeel force-pushed the pgdp-production branch 3 times, most recently from bbbce44 to 195502c Compare October 21, 2026 23:10
fixup Add redirects for old Page Browser pages
@cpeel
cpeel force-pushed the remove-final-require-login branch from 0c01ef7 to 4f1ec84 Compare October 26, 2026 18:37
@cpeel
cpeel merged commit 4c0263e into DistributedProofreaders:pgdp-production Oct 26, 2026
12 checks passed
@cpeel
cpeel deleted the remove-final-require-login branch October 26, 2026 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant