wplemon / gridd

Flexible grid-based WordPress theme
Other
39 stars 7 forks source link

FYI: Jetpack will be deprecating and removing the `jetpack_require_lib` function #205

Closed anomiex closed 2 years ago

anomiex commented 2 years ago

We're planning on deprecating and removing the jetpack_require_lib function from Jetpack, and a search of the WordPress Theme directory turned up your theme as the only one using it.

It looks like you're using existence of the function to determine if Jetpack is active. You might change your check to look for whether one of Jetpack's constants (such as JETPACK__VERSION or JETPACK__PLUGIN_DIR) is defined instead, or if the check is needed late enough you could check for class_exists( \Tonesque::class ) directly instead of trying to detect the specific plugin at all.

aristath commented 2 years ago

Thank you! I converted the checks to use the constant 👍