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

Feature/detect classic editor posts #53

Merged
merged 3 commits into from
May 13, 2024

Conversation

dsawardekar
Copy link
Collaborator

Description of the Change

This PR adds support for detecting Classic Editor blocks mixed with Gutenberg Blocks. This allows Block Catalog to find both legacy Classic Editor posts and posts that have other Gutenberg blocks alongside Classic Editor content.

How to test the Change

Run the Block Catalog index with content that uses Classic Editor or has hybrid content. Or view the blocks for a post via CLI. You should see a Core > Classic block for that post in the block catalog taxonomy.

Output for a Classic Editor Post & a Hybrid Post:
image

image

Changelog Entry

Added - Classic Editor block detection

Credits

Props @dsawardekar

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

@dsawardekar
Copy link
Collaborator Author

Note: This PR is branched off #52 as it depends on the earlier work. Here is the diff of the changes,

feature/add-export-command...feature/detect-classic-editor-posts

@dsawardekar dsawardekar marked this pull request as ready for review May 7, 2024 04:57
@dsawardekar dsawardekar requested a review from a team as a code owner May 7, 2024 04:57
@dsawardekar dsawardekar requested review from Sidsector9 and removed request for a team May 7, 2024 04:57
@jeffpaul jeffpaul added this to the 1.6.0 milestone May 7, 2024
@jeffpaul jeffpaul requested a review from peterwilsoncc May 7, 2024 13:15
Sidsector9
Sidsector9 previously approved these changes May 8, 2024
Copy link
Member

@Sidsector9 Sidsector9 left a comment

Choose a reason for hiding this comment

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

Works as expected 👍

Merge #52 before this PR as there are some changes requested in 52.

@dsawardekar
Copy link
Collaborator Author

@Sidsector9 I've fixed that issue, can you take another look? thx

@jeffpaul jeffpaul merged commit ee4125b into develop May 13, 2024
6 checks passed
@jeffpaul jeffpaul deleted the feature/detect-classic-editor-posts branch May 13, 2024 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants