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

Jetpack Post by Email feature: numeric-only tags are not processed correctly #40570

Open
alinclamba opened this issue Dec 11, 2024 · 3 comments
Labels
[Experiment] AI labels added [Feature Group] Content Management Features related to the tools and screens that admins use to manage their sites core content. [Feature] Post By Email [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] Low [Status] Auto-allocated [Status] Escalated to Product Ambassadors [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack Triaged [Type] Bug When a feature is broken and / or not performing as intended

Comments

@alinclamba
Copy link

Impacted plugin

Jetpack

Quick summary

Numeric-only tags (e.g., [tags 3322]) are not processed correctly in the Post by Email feature. Alphanumeric tags (e.g., [tags pp3322]) work as expected.

Steps to reproduce

  1. Enable the Post by Email feature
  2. Compose an email and include a numeric tag (e.g., [tags 3322]).
  3. Send the email to be published.
  4. Check the post once published and see that the tag is not displayed
  5. Compose another email
  6. Add an alphanumeric tag this time (e.g., [tags pp3322]) and send it.
  7. Check the post published and see that the tag was correctly processed and displayed

A clear and concise description of what you expected to happen.

Issue was reported by a user in the forum, here: https://wordpress.org/support/topic/jetpack-post-by-email-tag-shortcode/ but I was able to replicate this on my test site.

What actually happened

No response

Impact

Some (< 50%)

Available workarounds?

No but the platform is still usable

If the above answer is "Yes...", outline the workaround.

No response

Platform (Simple and/or Atomic)

No response

Logs or notes

No response

@alinclamba alinclamba added [Type] Bug When a feature is broken and / or not performing as intended Needs triage Ticket needs to be triaged labels Dec 11, 2024
@github-actions github-actions bot added [Status] Escalated to Product Ambassadors [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] High [Feature Group] Content Management Features related to the tools and screens that admins use to manage their sites core content. [Feature] Jetpack [Feature] Post By Email labels Dec 11, 2024
Copy link
Contributor

OpenAI suggested the following labels for this issue:

  • [Feature Group] Content Management: The issue relates to managing content through the Post by Email feature of the Jetpack plugin.
  • [Feature] Post By Email: This label is directly related to the specific feature of Jetpack that is experiencing the issue.
  • [Feature] Jetpack: Since the issue occurs within the Jetpack plugin itself, this label is appropriate.

@matticbot matticbot added the [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack label Dec 11, 2024
@Robertght
Copy link

📌 REPRODUCTION RESULTS

  • Tested on Simple – Could Not Replicate
  • Tested on Atomic – Uncertain
  • Replicable outside of Dotcom – Yes

📌 FINDINGS/SCREENSHOTS/VIDEO

  • Weirdly, the post had the numeric tag if I included more than one numeric tag and some non-numeric ones.
  • On Simple sites, the numeric tags are displayed in the post settings correctly

📌 ACTIONS

  • Triaged

@Robertght Robertght added Triaged and removed Needs triage Ticket needs to be triaged labels Dec 11, 2024
@Robertght Robertght moved this from Needs Triage to Triaged in Automattic Prioritization: The One Board ™ Dec 11, 2024
@kraftbj
Copy link
Contributor

kraftbj commented Dec 11, 2024

This is due to this:

* Consequence: To add a category/tag whose name is '123', the client must

The PBE processor takes the tags in the shortcode and passes them to the v1 WP.com new post endpoint, which accepts tag names or tags ID. If it is numeric only, it assume it is a term ID.

We'd need to either change PBE to take a numeric tag and look up the corresponding term ID, using that instead, when passing it to the API or something with the endpoint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Experiment] AI labels added [Feature Group] Content Management Features related to the tools and screens that admins use to manage their sites core content. [Feature] Post By Email [Plugin] Jetpack Issues about the Jetpack plugin. https://wordpress.org/plugins/jetpack/ [Pri] Low [Status] Auto-allocated [Status] Escalated to Product Ambassadors [Status] Priority Review Triggered The guild in charge of triage has been notified of this issue in Slack Triaged [Type] Bug When a feature is broken and / or not performing as intended
Projects
Development

No branches or pull requests

5 participants