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

Divi Layout block not working with Learning Mode + Yoast SEO. #6112

Open
aaronfc opened this issue Nov 14, 2022 · 2 comments
Open

Divi Layout block not working with Learning Mode + Yoast SEO. #6112

aaronfc opened this issue Nov 14, 2022 · 2 comments
Assignees
Labels
[Pri] Low Cosmetic issue, low impact, easy workaround [Status] Needs Triage Issue needs to be triaged Third-Party [Type] Bug

Comments

@aaronfc
Copy link
Contributor

aaronfc commented Nov 14, 2022

Reported by Gonzalo.

Steps to Reproduce

  1. Install Divi theme and enable.
  2. Install YOAST SEO plugin and enable.
  3. Enable Learning Mode on a Course (or full site).
  4. Create a Lesson (linked to a Course with Learning Mode enabled if not enabled site wide).
  5. Add the "Divi Layout" block to the Lesson.
  6. Click on the "Start building" main button.

What I Expected

Divi layout builder to load correctly.

What Happened Instead

Divi layout builder stuck in loading screen.

PHP / WordPress / Sensei LMS version

WP: 6.1
Sensei Pro (WC Paid Courses): 4.8.1.1.8.1

Screenshot / Video

Screen.Capture.on.2022-11-14.at.11-08-54.mov

Context / Source

  • Disabling Yoast SEO plugin fixes the issue.
  • Disabling Learning Mode in the Course or site fixes the issue.
  • I saw some javascript errors but they do not seem related (still appear when the issue is not reproducible by not having Learning Mode enabled or Yoast SEO).
  • This is preventing the usage of Divi blocks inside the Gutenberg editor. If we unblock this we make it easier for people to use Gutenberg editor more instead of Divi builder.
@aaronfc aaronfc added [Type] Bug [Status] Needs Triage Issue needs to be triaged labels Nov 14, 2022
@aaronfc
Copy link
Contributor Author

aaronfc commented Nov 15, 2022

Thanks to @renatho we saw this is linked to this previous/known issue: #5001

Apparently adding an excerpt to the Lesson (and saving) fixes the issue.

@renatho
Copy link
Contributor

renatho commented Jan 5, 2023

#6342 adds a temporary solution for this. So I'm changing the priority of this issue to low. The ideal would be to investigate it further in the future and provide a better solution.

@renatho renatho added the [Pri] Low Cosmetic issue, low impact, easy workaround label Jan 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Pri] Low Cosmetic issue, low impact, easy workaround [Status] Needs Triage Issue needs to be triaged Third-Party [Type] Bug
Projects
None yet
Development

No branches or pull requests

2 participants