Skip to content

Jetpack Sync: Sender: Return errors instead of plain false#7135

Merged
samhotchkiss merged 5 commits intomasterfrom
update/sync-sender-errors
May 8, 2017
Merged

Jetpack Sync: Sender: Return errors instead of plain false#7135
samhotchkiss merged 5 commits intomasterfrom
update/sync-sender-errors

Conversation

@lezama
Copy link
Copy Markdown
Contributor

@lezama lezama commented May 5, 2017

Improve debugging output for AT

@lezama lezama self-assigned this May 5, 2017
@lezama lezama added [Status] Needs Review This PR is ready for review. and removed [Status] In Progress labels May 5, 2017
Copy link
Copy Markdown
Contributor

@gravityrail gravityrail left a comment

Choose a reason for hiding this comment

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

LGTM!

// don't sync if importing
if ( defined( 'WP_IMPORTING' ) && WP_IMPORTING ) {
return false;
return new WP_Error( 'is_importing' );;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

extra semicolon

// don't sync if we are throttled
if ( $this->get_next_sync_time( $queue->id ) > microtime( true ) ) {
return false;
return new WP_Error( 'sync_throttled' );;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

extra semicolon

@lamosty
Copy link
Copy Markdown
Contributor

lamosty commented May 8, 2017

This is awesome, thanks!

@enejb
Copy link
Copy Markdown
Member

enejb commented May 8, 2017

❤️

@samhotchkiss samhotchkiss merged commit 90cfdf2 into master May 8, 2017
@samhotchkiss samhotchkiss removed the [Status] Needs Review This PR is ready for review. label May 8, 2017
samhotchkiss pushed a commit that referenced this pull request May 8, 2017
* Jetpack Sync: return errors, improves debugging :)

* missing ->id

* return new WP_Error( 'wpcom_error') whenever WPCOM fails

* fix tests

* remove extra semicolons
@enejb enejb deleted the update/sync-sender-errors branch May 9, 2017 16:18
jeherve added a commit that referenced this pull request May 11, 2017
@jeherve jeherve mentioned this pull request May 26, 2017
jeherve added a commit that referenced this pull request May 29, 2017
eliorivero pushed a commit that referenced this pull request May 30, 2017
* Changelog: first pass at a changelog for 5.0

* Changelog: delete 4.9 testing list.

* Changelog: update minimum WP version to match ver. in jetpack.php

Fixes #7158

* Changelog: add #6051

* Changelog: add #6753

* Changelog: add #6928

* Changelog: add #6964

* Changelog: add #7014

* Changelog: add #7057

* Changelog: add #7060

* Changelog: add #7068

* Changelog: add #7070

* Changelog: add #7072

* Changelog: add #7071

* Changelog: add release date and post shortlink.

* Changelog: add #7094

* Changelog: add #7100

* Changelog: add #7108

* Changelog: add #7113

* Changelog: add #7123

* Changelog: add #7135

* Changelog: add #7143

* Changelog: add #7151

* Changelog: add #6996

* Changelog: add #7105

* Changelog: add #7132

* Changelog: add #7166

* Changelog: fix typo in 4.9 changelog.

* Changelog: remove older releases' changelogs.

@see p1HpG7-42e-p2

* Changelog: add #7090

* Changelog: add #7095

* Changelog: add #7112

* Changelog: add #7115

* Changelog: add #7122

* Changelog: add #7137

* Changelog: add #7138

* Changelog: add #7140

* Changelog: add #7154

* Changelog: add ##7155

* Changelog: add #7163

* Changelog: add #7167

* Changelog: add #7171

* Changelog: add #7180

* Changelog: add #7181

* Changelog: add #7183

* Changelog: add #7184

* Changelog: add #7189

* Changelog: add #7191

* Changelog: add #7193

* Changelog: add #7198

* Changelog: add #7200

* Changelog: add #7209

* Changelog: add #7212

* Testing list: add instructions for #7115

* Changelog: add #7188

* Changelog: add #7205

* Changelog: add #7225

* Changelog: add #6872

* Changelog: add #7107

* Changelog: add #7118

* Changelog: add #7142

* Changelog: add #7170

* Changelog: add #7210

* Changelog: add #7218

* Changelog: add #7232

* Changelog: add #7211

* Changelog: add #7213

* Changelog: add #7229

* Changelog: add #7230

* Changelog: add #7214

* Draft changelog for 5.0

* Changelog updates: 2nd pass at a clearer changelog.

- Fix typos.
- Use consistent tense and tone across all changelog.
- Remove unclear items.

* Changelog: add #7026

* Changelog: add #7058

* Changelog: add #7125

* Changelog: add #7249

* Changelog: add #7185

* add mentions of image widget migration

* Changelog: add info about new output for CLI command.

* Changelog: add WP version number matching the new Image Widget.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants