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

Infinite Scroll: remove functions now included in Core, since 4.1 #3499

Merged
merged 1 commit into from
Mar 3, 2016

Conversation

jeherve
Copy link
Member

@jeherve jeherve commented Mar 3, 2016

See #1447

We can now remove wp_json_encode, _wp_json_sanity_check, _wp_json_convert_string
These 3 functions were added to Core in 4.1.
Jetpack now requires 4.3

See #1447

We can now remove `wp_json_encode`, `_wp_json_sanity_check`, `_wp_json_convert_string`
These 3 functions were added to Core in 4.1.
Jetpack now requires 4.3
@jeherve jeherve added [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Infinite Scroll [Team] tdiv [Status] Needs Review This PR is ready for review. labels Mar 3, 2016
@jeherve jeherve added this to the 3.9.3 milestone Mar 3, 2016
@dereksmart
Copy link
Member

👍

@dereksmart dereksmart 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 Mar 3, 2016
zinigor added a commit that referenced this pull request Mar 3, 2016
Infinite Scroll: remove functions now included in Core, since 4.1
@zinigor zinigor merged commit b106ed0 into master Mar 3, 2016
@zinigor zinigor deleted the remove/infinite-41-compat branch March 3, 2016 20:50
@zinigor zinigor removed the [Status] Ready to Merge Go ahead, you can push that green button! label Mar 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Infinite Scroll Touches WP.com Files [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants