Skip to content

Feature/detect classic editor posts#53

Merged
jeffpaul merged 3 commits intodevelopfrom
feature/detect-classic-editor-posts
May 13, 2024
Merged

Feature/detect classic editor posts#53
jeffpaul merged 3 commits intodevelopfrom
feature/detect-classic-editor-posts

Conversation

@dsawardekar
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator Author

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

@jeffpaul jeffpaul merged commit ee4125b into develop May 13, 2024
@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