WordPress-Android icon indicating copy to clipboard operation
WordPress-Android copied to clipboard

Published 20 hours ago • verified publisher icon wordpress-mobile

YouTube shortcode not rendering correctly in notification mention

Open guarani opened this issue 5 years ago • 4 comments

Expected behavior

When viewed via the Notifications tab in WPAndroid, a comment that includes a Shortcode block which in-turn contains the YouTube shortcode (e.g. [youtube https://www.youtube.com/watch?v=fX8YLri4wxU]) should be rendered correctly (i.e. the YouTube video should be visible in the comment). Note: this works fine for posts that contain this same content.

Actual behavior

The raw shortcode itself is visible instead of the referenced YouTube video.

Post notification — works here ✅ Comment notification — doesn't work here ❌
Screenshot showing YouTube shortcode rendering correctly in post notification in WPAndroid Screenshot showing YouTube shortcode rendering incorrectly in comment notification in WPAndroid

Steps to reproduce the behavior

  1. On the web, log into a WordPress site that has Gutenberg enabled in comments (e.g. create a site on http://wp.com/p2)
  2. Add another WordPress user, granting the Administrator role (other roles may work)
  3. Still on the same site, create a reply (comment) to a post, @-mentioning the newly added user
  4. Add an Shortcode block to that same reply (sample content [youtube https://www.youtube.com/watch?v=fX8YLri4wxU])
  5. Send the reply
  6. From the WPAndroid app, log in as the newly added Administrator
  7. In the Notifications tab, locate the notification for the reply that @-mentioned the user
  8. Open the comment, and notice the video is not visible — only the raw shortcode is visible
Tested on Samsung S10, Android 10, WPAndroid 15.4.1

guarani avatar Aug 26 '20 00:08 guarani

This seems still to be current in develop (at this point in history).

Tested on Pixel 4 Emu API29.

develric avatar Apr 19 '21 22:04 develric

This issue has been marked as stale because:

  • It has been inactive for the past year.
  • It isn't in a project or a milestone.
  • It hasn’t been labeled [Pri] Blocker, [Pri] High, or good first issue.

Please comment with an update if you believe this issue is still valid or if it can be closed. This issue will also be reviewed for validity and priority during regularly scheduled triage sessions.

stale[bot] avatar Apr 28 '22 06:04 stale[bot]

I confirmed that this is still an issue both in the notifications and the reader

Notification detail Reader
Screenshot_20240308_160207 Screenshot_20240308_160244

antonis avatar Mar 08 '24 14:03 antonis

Thanks for reporting! 👍

dangermattic avatar Mar 08 '24 14:03 dangermattic