-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve error messages when logging into self-hosted sites #17724
Conversation
You can trigger an installable build for these changes by visiting CircleCI here. |
You can trigger optional UI/connected tests for these changes by visiting CircleCI here. |
2418450
to
56f132e
Compare
56f132e
to
4e63907
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and working great! I also tested with the plugin set to "Off" mode and login to a self-hosted site worked as normal.
Code changes look good too!
e7153a1
to
0f655c7
Compare
0f655c7
to
9706eeb
Compare
Fixes #17119
Related PRs:
To test:
On a self-hosted site install the attached plugin (XML-RPC Tweak) which will aid in testing: xml-rpc-tweak.zip
Some steps will require the tester to log in using an invalid username and or password. This is only to overcome a limitation with how the plugin simulates certain failures.
Test 1:
Test 2:
Test 3:
Test 4:
Regression Notes
PR submission checklist:
RELEASE-NOTES.txt
if necessary.