Skip to content

Improve logging for Jetpack REST migrations#3731

Merged
sendhil merged 6 commits intodevelopfrom
issue/3627-jetpack-rest-add-migration-logging
May 29, 2015
Merged

Improve logging for Jetpack REST migrations#3731
sendhil merged 6 commits intodevelopfrom
issue/3627-jetpack-rest-add-migration-logging

Conversation

@koke
Copy link
Member

@koke koke commented May 27, 2015

As part of #3627 this improves logging so we can support Jetpack sites better:

  • It will log when a new site is added automatically via syncBlogs
  • It will log when a Jetpack site is migrated to use REST
  • Improves the blog list logged on launch

Some sample blogs and their logged description on launch

Before

Name: Edge test site URL: http://edge.wpmt.co XML-RPC: http://edge.wpmt.co/xmlrpc.php isWpCom: NO blogId: 0 jetpackAccount: PRESENT
Name: Jetpack Test #2 URL: http://jp2.koke.co XML-RPC: http://jp2.koke.co/xmlrpc.php isWpCom: NO blogId: 0 jetpackAccount: NONE
Name: Jorge Bernal URL: http://jorgebernal.es XML-RPC: https://jorgebernales.wordpress.com/xmlrpc.php isWpCom: YES blogId: 17691640 jetpackAccount: NONE
Name: Jorge Bernal URL: http://jorgebernal.info XML-RPC: https://jorgebernalen.wordpress.com/xmlrpc.php isWpCom: YES blogId: 17758711 jetpackAccount: NONE
Name: Koke's Journal URL: http://koke.me XML-RPC: https://kokejournal.wordpress.com/xmlrpc.php isWpCom: YES blogId: 16764956 jetpackAccount: NONE
Name: Test site URL: http://test.koke.co XML-RPC: http://test.koke.co/xmlrpc.php isWpCom: YES blogId: 63902892 jetpackAccount: NONE
Name: WordPress Latest URL: http://wp.koke.me XML-RPC: http://wp.koke.me/xmlrpc.php isWpCom: NO blogId: 0 jetpackAccount: NONE
Name: WordPress Secure URL: http://wps.koke.me XML-RPC: https://wps.koke.me/xmlrpc.php isWpCom: YES blogId: 91395574 jetpackAccount: NONE
Name: WordPress Trunk URL: http://wpt.koke.me XML-RPC: http://wpt.koke.me/xmlrpc.php isWpCom: NO blogId: 0 jetpackAccount: NONE

After

<Blog Name: Edge test site URL: http://edge.wpmt.co XML-RPC: http://edge.wpmt.co/xmlrpc.php jetpack: 🚀🚀 Jetpack 3.5.3 fully connected as wpmtestjp with site ID 59803729
<Blog Name: Jetpack Test #2 URL: http://jp2.koke.co XML-RPC: http://jp2.koke.co/xmlrpc.php jetpack: 🚀❌ Jetpack 3.5.3 not connected
<Blog Name: Jorge Bernal URL: http://jorgebernal.es XML-RPC: https://jorgebernales.wordpress.com/xmlrpc.php wp.com account: kokeme blogId: 17691640
<Blog Name: Jorge Bernal URL: http://jorgebernal.info XML-RPC: https://jorgebernalen.wordpress.com/xmlrpc.php wp.com account: kokeme blogId: 17758711
<Blog Name: Koke's Journal URL: http://koke.me XML-RPC: https://kokejournal.wordpress.com/xmlrpc.php wp.com account: kokeme blogId: 16764956
<Blog Name: Test site URL: http://test.koke.co XML-RPC: http://test.koke.co/xmlrpc.php wp.com account: kokeme blogId: 63902892
<Blog Name: WordPress Latest URL: http://wp.koke.me XML-RPC: http://wp.koke.me/xmlrpc.php jetpack: 🚀❔Jetpack not installed
<Blog Name: WordPress Secure URL: http://wps.koke.me XML-RPC: https://wps.koke.me/xmlrpc.php wp.com account: kokeme blogId: 91395574
<Blog Name: WordPress Trunk URL: http://wpt.koke.me XML-RPC: http://wpt.koke.me/xmlrpc.php jetpack: 🚀✅ Jetpack 3.5.3 connected as UNKNOWN with site ID 22793269

Needs Review: @sendhil

@koke koke mentioned this pull request May 28, 2015
12 tasks
Copy link
Contributor

Choose a reason for hiding this comment

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

Cute use of emoji ;)

@sendhil
Copy link
Contributor

sendhil commented May 29, 2015

Looks good to me.

sendhil added a commit that referenced this pull request May 29, 2015
…t-add-migration-logging

Improve logging for Jetpack REST migrations
@sendhil sendhil merged commit a73dc8d into develop May 29, 2015
@sendhil sendhil deleted the issue/3627-jetpack-rest-add-migration-logging branch May 29, 2015 23:07
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.

2 participants