Skip to content

Sync post list 1.1 endpoint.#7008

Merged
zinigor merged 3 commits intofeature/sync-json-endpointsfrom
sync/list-posts-1-1-endpoint
Apr 28, 2017
Merged

Sync post list 1.1 endpoint.#7008
zinigor merged 3 commits intofeature/sync-json-endpointsfrom
sync/list-posts-1-1-endpoint

Conversation

@zinigor
Copy link
Copy Markdown
Contributor

@zinigor zinigor commented Apr 20, 2017

No description provided.

@zinigor zinigor added the [Status] Needs Review This PR is ready for review. label Apr 20, 2017
@zinigor
Copy link
Copy Markdown
Contributor Author

zinigor commented Apr 21, 2017

Master issue: #7026

@zinigor zinigor changed the base branch from master to feature/sync-json-endpoints April 21, 2017 16:08
}

if ( defined( 'IS_WPCOM' ) && IS_WPCOM ) {
if ( !isset( $return[$key] ) )
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we take that opportunity to clean up that code so it respects the WP coding standards?

Copy link
Copy Markdown
Contributor

@eliorivero eliorivero left a comment

Choose a reason for hiding this comment

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

🐑

@eliorivero eliorivero added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Apr 24, 2017
if ( defined( 'IS_WPCOM' ) && IS_WPCOM ) {
if ( !isset( $return[$key] ) )
$return[$key] = new stdClass;
$return[$key]->wpcom = true;
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.

Although it's not a blocker @zinigor the spacing could be fixed in both sides.

@zinigor
Copy link
Copy Markdown
Contributor Author

zinigor commented Apr 28, 2017

Appreciate the comments, but I'd rather sync this now, and then fix spacing on a file that has a common ancestor. We'll get this all fixed!

@zinigor zinigor merged commit 756462e into feature/sync-json-endpoints Apr 28, 2017
@zinigor zinigor removed the [Status] Ready to Merge Go ahead, you can push that green button! label Apr 28, 2017
@jeherve jeherve deleted the sync/list-posts-1-1-endpoint branch April 28, 2017 15:30
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.

4 participants