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

Update apiVersion of Recommended Content Block to version 3 #769

Open
fabiankaegy opened this issue May 15, 2024 · 4 comments
Open

Update apiVersion of Recommended Content Block to version 3 #769

fabiankaegy opened this issue May 15, 2024 · 4 comments
Labels
type:enhancement New feature or request.
Milestone

Comments

@fabiankaegy
Copy link
Member

The recommended content block currently uses apiVerson: 2. In order to not prevent adoption of the iFramed post editor this should get updated to apiVersion: 3.

Version 3 (>= WordPress 6.3)
The post editor will be iframed if all registered blocks have a Block API version 3 or higher and the editor has no classic meta boxes below the blocks. Adding version 3 support means that the block should work inside an iframe, though the block may still be rendered outside the iframe if not all blocks support version 3.

@fabiankaegy fabiankaegy added the type:enhancement New feature or request. label May 15, 2024
@jeffpaul jeffpaul added this to the 3.1.0 milestone May 15, 2024
@jeffpaul jeffpaul moved this from Incoming to To Do in Open Source Practice May 15, 2024
@dkotter
Copy link
Collaborator

dkotter commented May 15, 2024

I'm not opposed to making this change but since this feature is deprecated (due to no longer being supported by Microsoft) I'd recommend we don't spend much time on this. If all we need to due is bump the apiVersion to 3, great, but if there are additional things that need changed to ensure things work, may want to just leav this alone as this code will be removed eventually anyway

@fabiankaegy
Copy link
Member Author

The issue we are running into is that if any block, regardless of whether it is in use or not, has an apiVersion smaller than 3 that means that the post editor doesn't get rendered in an iframe... So even with it being deprecated. As long as it exists it blocks the iframe

@dkotter
Copy link
Collaborator

dkotter commented May 15, 2024

The issue we are running into is that if any block, regardless of whether it is in use or not, has an apiVersion smaller than 3 that means that the post editor doesn't get rendered in an iframe... So even with it being deprecated. As long as it exists it blocks the iframe

Hmm.. interesting, that's fun. I imagine though it's only an issue if the block is actually registered and available to select from, correct? If so, that's only the case here if the Recommended Content Feature is turned on which I don't have numbers on that but I imagine it's a fairly small amount of people that actually set this up before it was deprecated.

But again, not opposed to bumping that version, I just know in testing in another plugin, that version bump broke the block and I'm concerned that may be the case here as well, which would require more effort on a feature that we aren't currently supporting.

@fabiankaegy
Copy link
Member Author

If so, that's only the case here if the Recommended Content Feature is turned on which I don't have numbers on that but I imagine it's a fairly small amount of people that actually set this up before it was deprecated.

Ahhh okay yeah if this is behind a feature flag without a lot of use this is not as big a concern :)

@dkotter dkotter modified the milestones: 3.1.0, 3.2.0 Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement New feature or request.
Projects
Status: To Do
Development

No branches or pull requests

3 participants