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

feat: parse meta attributes #286

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Conversation

justlevine
Copy link
Contributor

@justlevine justlevine commented Sep 17, 2024

What

This PR adds support for attributes with a meta source.

Important

This PR is based on #284 , which should be merged first.

Relevant diff: 0f3010c

Why

Although meta is considered deprecated, many third-party and legacy blocks still rely on in.

Additional Notes

Tests will be backfilled alongside other ongoing work as they require mocking a custom block with a meta field.

@justlevine justlevine requested a review from a team as a code owner September 17, 2024 15:41
Copy link

changeset-bot bot commented Sep 17, 2024

🦋 Changeset detected

Latest commit: 9e94b89

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@wpengine/wp-graphql-content-blocks Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@justlevine
Copy link
Contributor Author

Rebased post #279

@theodesp theodesp merged commit d123b81 into wpengine:main Sep 23, 2024
10 of 11 checks passed
@justlevine justlevine deleted the feat/meta-attributes branch September 23, 2024 12:46
@justlevine justlevine restored the feat/meta-attributes branch September 23, 2024 15:06
@justlevine
Copy link
Contributor Author

This was dependent on #284 . I'll rebase that PR to fix the issues surfaced there by @jasonbahl shortly

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