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

[MWPW-162639] youtube block a tag fix #3703

Open
wants to merge 4 commits into
base: stage
Choose a base branch
from
Open

Conversation

DKos95
Copy link
Contributor

@DKos95 DKos95 commented Feb 19, 2025

This resolves the issue where the youtube link was being processed into a block because the link was first in the paragraph and not text.

Resolves: MWPW-162639

Test URLs:

@DKos95 DKos95 requested a review from a team February 19, 2025 10:21
Copy link
Contributor

aem-code-sync bot commented Feb 19, 2025

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link
Contributor

aem-code-sync bot commented Feb 19, 2025

Page Scores Audits Google
📱 /drafts/cpeyer/temp/og-issue?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /drafts/cpeyer/temp/og-issue?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@aem-code-sync aem-code-sync bot temporarily deployed to inline-youtube-link February 19, 2025 13:04 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to inline-youtube-link February 20, 2025 11:11 Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to inline-youtube-link February 20, 2025 11:12 Inactive
@DKos95 DKos95 force-pushed the inline-youtube-link branch from 8847b04 to 2fe450f Compare February 20, 2025 11:43
@SilviuLCF SilviuLCF requested a review from zagi25 February 21, 2025 09:29
Copy link
Contributor

Reminder to set the Ready for Stage label - to queue this to get merged to stage & production.

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.

3 participants