Skip to content

Require authentication to access project pages - #1526

Merged
cpeel merged 2 commits into
DistributedProofreaders:masterfrom
cpeel:auth-project-pages
Sep 19, 2025
Merged

cpeel merged 2 commits into
DistributedProofreaders:masterfrom
cpeel:auth-project-pages

Conversation

@cpeel

@cpeel cpeel commented Sep 1, 2025

Copy link
Copy Markdown
Member

This PR implements this proposal by the GM for project pages to require authentication. Because the project pages are the only thing of substance we have in our sitemap, this also removes the sitemap.

The changes to project.php are best viewed ignoring whitespace as most of the changes are removing conditionals.

This PR will merge in no earlier than after the R202509 release and not until the GM gives the OK after she gathers community input and consults with the squirrels.

Sandbox: https://www.pgdp.org/~cpeel/c.branch/auth-project-pages/

We're locking down the project pages which makes the sitemap no
longer useful.
@cpeel cpeel self-assigned this Sep 1, 2025
@cpeel
cpeel merged commit e0464b1 into DistributedProofreaders:master Sep 19, 2025
9 checks passed
@cpeel
cpeel deleted the auth-project-pages branch September 19, 2025 21:41
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.

4 participants