Skip to content
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

Show meaningful error when xmlrpc broken #10308

Merged

Conversation

ScoutHarris
Copy link
Contributor

Fixes #10293

NOTE: This uses an updated WordPressKit pod. I will update the Podfile to a valid version after it's been approved.

Related WordPressKit PR: wordpress-mobile/WordPressKit-iOS#48

To test:


  • On the login page, select Log in by entering your site address.
  • Enter an address that is not a WordPress site.
  • Verify the message is now Couldn't connect to the WordPress site. There is no valid WordPress site at this address. Check the site address (URL) you entered.

not_wp_site


  • On the login page, select Log in by entering your site address.
  • Enter an address of a WP site that has RSD set up but XML-RPC is not.
    • Feel free to ping me or @SiobhyB for an example customer site if needed.
  • Verify the message is now Couldn't connect. Required XML-RPC methods are missing on the server.

xmlrpc_broken

Copy link
Member

@aerych aerych left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commented on wordpress-mobile/WordPressKit-iOS#48
:shipit: with the updated pod :)

@ScoutHarris ScoutHarris merged this pull request into develop Oct 24, 2018
@ScoutHarris ScoutHarris deleted the fix/10293-show_meaningful_error_when_xmlrpc_broken branch October 24, 2018 19:16
jkmassel pushed a commit that referenced this pull request Oct 25, 2018
…gful_error_when_xmlrpc_broken

Show meaningful error when xmlrpc broken
jtreanor pushed a commit that referenced this pull request Nov 20, 2018
…gful_error_when_xmlrpc_broken

Show meaningful error when xmlrpc broken
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants