Please stop anonymous login to edit.php.net

Sergey Panteleev <[email protected]>
Newsgroups gmane.comp.php.documentation.general
Message-ID <AM9PR09MB50286B08B7E9D6352D79D9C3AE429@AM9PR09MB5028.eurprd09.prod.outlook.com>
Done :)

From: Christoph M. Becker <[email protected]>
Date: Monday, 26 April 2021, 14:07
To: Sergey Panteleev <[email protected]>
Subject: Re: Please stop anonymous login to edit.php.net
Hi Sergey!

On 26.04.2021 at 12:48, Sergey Panteleev wrote:

> I'm now trying to implement the integration of Github and the editor,
> but I don’t yet understand how to do it correctly.
>
> At the moment, the patch can be downloaded as a .diff file and applied locally,
> how to send it to GitHub on behalf of the user without compromising security —
> I haven’t figured it out yet.

I don't think that makes sense.  Instead users should provide PRs in the
first place; the Github editor[1] makes that pretty easy (although it's
far from perfect for translation updates).

> Basically, the editor now works as a tool for tracking changes
> (as http://doc.php.net/revcheck.php doesn't work yet).
>
> If we disable the login for non-PHP users — is there any point then in the editor as such?

I heard that some translators prefer using edit.php.net, and I think
there is no reason to disallow that.  It is just that I feel we
shouldn't invite users to provide patches via edit.php.net, if we don't
address them, and GH PRs also have the advantage that anybody can easily
comment on a PR which appears to be helpful.  Furthermore, you can
subscribe to PRs or even the repo.

If you just forgot to sent the reply to the list, feel free to forward
my reply. :)

Cheers,
Christoph

[1]
<https://docs.github.com/en/github/managing-files-in-a-repository/editing-files-in-your-repository>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.