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

fix: Remove remove_actions() workaround. #70

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

ashutoshgautams
Copy link
Contributor

@ashutoshgautams ashutoshgautams commented Feb 6, 2025

What

Remove temporary workaround for wp-graphql-content-blocks plugin conflicts

Why

The workaround is no longer needed as the underlying issue has been fixed in wp-graphql-content-blocks PR #262

Related Issue(s):

Part of https://github.com/rtCamp/headless/issues/303 (Task sheet)

How

  • Removed remove_actions() method and its docblock
  • Removed call to $this->remove_actions()

Testing Instructions

  1. Run PHPUnit tests
  2. Verify all tests pass without errors

Screenshots

N/A

Additional Info

N/A

Checklist

  • I have read the Contribution Guidelines.
  • My code is tested to the best of my abilities.
  • My code passes all lints (PHPCS, PHPStan, ESLint, etc.).
  • My code has detailed inline documentation.
  • I have added unit tests to verify the code works as intended.
  • I have updated the project documentation accordingly.

@ashutoshgautams ashutoshgautams self-assigned this Feb 6, 2025
@justlevine justlevine merged commit c7b4e5c into rtCamp:develop Feb 6, 2025
11 checks passed
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.

2 participants