Skip to content

Remove final require_login() - #1685

Open
cpeel wants to merge 29 commits into
DistributedProofreaders:pgdp-productionfrom
cpeel:remove-final-require-login
Open

cpeel wants to merge 29 commits 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 and others added 28 commits October 16, 2026 10:15
This marks the first commit of the pgdp.net-specific production
hacks. Update the README to indicate that this branch should not
be merged into master.
And quell (correct) PHPStan warning (DistributedProofreaders#1087)
The dp_branding directory is outside the `/c` directory structure.
Add it to the pgdp-production branch as a local hack so that it is
available on both PROD and TEST, and can be updated in both places
as part of the code deployment process and not have someone have to
remember to make sure to update both places when updates are made.
In pgdp-production, users must first pass a quiz and request access
before they can proofreading in P1.

We also need to skip ProjectTest::test_can_be_proofed_available
as it no longer passes with this update.
GM request -- update the introductory information for the specialized
quizzes explicitly stating that they are not required.
The GM requested that the paragraph concerning volunteer service
verification be expanded and updated. The original para has been
removed from `master`. This PR adds the expanded para into the
`pgdp-production` branch.
This completes the move of the acces change emails from `master` to
`pgdp-production`.
Allow Project Gutenberg's pg-automation account to view clearances.
PROD has already been hotfixed; this PR is to get the change into
the site-specific code.
As requested by GM, update newcomer reserve to 14 days.
Don't encourage users to apply for F2 and P3 as those require more
handholding and review.
At the request of the Portuguese PMs and approved by the GM,
remove the daily page limit for Portuguese
fixup Add redirects for old Page Browser pages
@cpeel
cpeel force-pushed the remove-final-require-login branch from b10f8aa to 0c01ef7 Compare October 16, 2026 17:17
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.

2 participants