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

Subscriptions: Remove the shortcode #15227

Closed
scruffian opened this issue Apr 1, 2020 · 10 comments
Closed

Subscriptions: Remove the shortcode #15227

scruffian opened this issue Apr 1, 2020 · 10 comments
Assignees
Labels
[Block] Subscriptions [Focus] Blocks Issues related to the block editor, aka Gutenberg, and its extensions developed in Jetpack [Focus] FSE Issues related to the site editor / Full Site Editing / FSE feature in Gutenberg [Pri] High [Size] M [Type] Task

Comments

@scruffian
Copy link
Member

scruffian commented Apr 1, 2020

The Subscriptions block outputs a shortcode.

Describe the solution you'd like

We should remove this, and output the markup we want for the block directly as suggested by @mtias

Describe alternatives you've considered

Keeping the shortcode means that people who use the shortcode continue to be able to benefit from the additional features we add, but this results in more complex development and doesn't encourage people to move to Gutenburg.

@scruffian scruffian added Simple Payments [Focus] Blocks Issues related to the block editor, aka Gutenberg, and its extensions developed in Jetpack [Block] Subscriptions labels Apr 1, 2020
@jeherve jeherve added the [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it label Apr 1, 2020
@creativecoder creativecoder added [Pri] Normal [Type] Task [Size] M and removed [Type] Enhancement Changes to an existing feature — removing, adding, or changing parts of it labels Apr 30, 2020
@mmtr mmtr changed the title Simple Payments and Subscriptions: Remove the shortcode Subscriptions: Remove the shortcode Sep 29, 2020
@mmtr
Copy link
Member

mmtr commented Sep 29, 2020

We created a separated issue for stopping using the simple-payment shortcode as output of the "Pay with PayPal" block: #17246 (fixed by #17256).

I've updated this issue to reflect we only need to handle the Subscriptions block now.

@simison
Copy link
Member

simison commented Sep 29, 2020

Thanks for this! Leaving the shortcode in Simple payments block was one of my biggest regrests. 😅

It came from wish to not add yet another simple payments implementation (alongside widget, classic editor block, and shortcode) but it turned out to be short-sighted and caused more trouble than it was aiming to save time.

@coreymckrill
Copy link
Contributor

Noting that, at present, shortcodes included in the block templates and block template parts of FSE themes are not rendered, and that probably won't change. It seems like that might make this issue a bit more urgent.

@simison
Copy link
Member

simison commented Sep 28, 2021

@kraftbj @jeherve 👋 this issue should be tracked for WP 5.9 readiness.

@jeherve
Copy link
Member

jeherve commented Sep 30, 2021

this issue should be tracked for WP 5.9 readiness.

Could you clarify a bit what you mean by that? Would 5.9 break rendering of shortcodes in some way?

@jeherve
Copy link
Member

jeherve commented Oct 7, 2021

ah, I was able to reproduce. For some reason, the shortcode is rendered when used in a footer template, but not in a single post template. I added this issue to the primary issue for WP 5.9 compat.

@simison
Copy link
Member

simison commented Oct 7, 2021

@jeherve is there a way to tell which other blocks output a shortcode? I'm thinking forms and simple payments from memory. Anything else?

@scruffian
Copy link
Member Author

Maybe the contact form too?

@jeherve
Copy link
Member

jeherve commented Oct 7, 2021

I'm thinking forms and simple payments from memory.

There are no other blocks that should be problematic in that regard as far as I know.

@jeherve jeherve added the [Focus] FSE Issues related to the site editor / Full Site Editing / FSE feature in Gutenberg label Jul 15, 2022
@TimBroddin
Copy link
Contributor

This is fixed with #26947

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Block] Subscriptions [Focus] Blocks Issues related to the block editor, aka Gutenberg, and its extensions developed in Jetpack [Focus] FSE Issues related to the site editor / Full Site Editing / FSE feature in Gutenberg [Pri] High [Size] M [Type] Task
Projects
None yet
Development

No branches or pull requests

9 participants