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

DevEx API - WordPress-y version #195

Open
wants to merge 1 commit into
base: devex_api
Choose a base branch
from
Open

DevEx API - WordPress-y version #195

wants to merge 1 commit into from

Conversation

ashfame
Copy link
Member

@ashfame ashfame commented Jan 2, 2025

This PR offers a WordPress-y API for developers which leverages the Ops class itself introduced in #192 (and requires that PR be merged first)

This version is better than presented in #125 since this controls preview automatically.

To test this, I replaced the Ops::handle() calls from Transformer class' constructor and used:

add_action( 'data_liberated_' . SubjectType::BLOGPOST->value, array( $this, 'handler' ) );
add_action( 'data_liberated_' . SubjectType::PAGE->value, array( $this, 'handler' ) );

to achieve the same effect.

Closes #194

@ashfame ashfame requested a review from psrpinto January 2, 2025 18:05
@ashfame ashfame self-assigned this Jan 2, 2025
@ashfame ashfame linked an issue Jan 2, 2025 that may be closed by this pull request
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.

Build WordPress-y interface on top of our interface
1 participant