Page MenuHomePhorge

PhabricatorPhabricatorAuthProvider: Remove unnecessary Remarkup
ClosedPublic

Authored by aklapper on May 19 2023, 15:13.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 06:19
Unknown Object (File)
Wed, Apr 10, 21:39
Unknown Object (File)
Sun, Apr 7, 05:17
Unknown Object (File)
Mon, Apr 1, 01:43
Unknown Object (File)
Mon, Apr 1, 01:43
Unknown Object (File)
Mon, Apr 1, 01:42
Unknown Object (File)
Fri, Mar 29, 06:33
Unknown Object (File)
Tue, Mar 26, 08:37

Details

Summary

This is a small fix in a specific user message example. See other messages in the very same file which also use <tt>.

Here the problematic phrase:

Auth screenshot Remarkup problem.png (204×759 px, 24 KB)

See the example in the phrase "Use lowercase letters, digits, and periods. For example: ...".

Test Plan

After this change, the specific message "The URI where the OAuth server is installed. For example" from the page /auth/config/edit/?provider=PhabricatorPhabricatorAuthProvider does not have anymore backticks shown to the end user.

Diff Detail

Repository
rP Phorge
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

(I have a déjà vu about this patch, I do not know why)

Thanks so much. Just for more context, here the fixed thing:

Phorge Auth fixed.png (259×1 px, 32 KB)

Tested locally following the test plan, nothing explodes.

To test that, I had to destroy my local Auth configuration to have a clean environment. I've done that with:

DELETE FRom auth_providerconfig WHERE providerClass != 'PhabricatorPasswordAuthProvider';

lgtm

This revision is now accepted and ready to land.May 19 2023, 18:24