- Issue created by @furamag
- πΊπΈUnited States jrockowitz Brooklyn, NY
Generally, most sites won't want or need Dynamic Yield for admin pages, and there should be a checkbox to enable/disable Dynamic Yield for admin pages. I am not sure we need to support custom exclude rules for the public-facing page.
- πΊπΈUnited States fathershawn New York
We already to not load DY on admin routes. The presenting issue is that some pages often use the admin theme, such as entity edit forms. This seemed like a simple way to exclude certain path patterns.
We could programmatically detect the active theme, get then name of the admin theme and compare to exclude DY.
- πΊπΈUnited States fathershawn New York
What ever we implement, if there is configuration we should collect in via
\Drupal\dynamic_yield\Form\FeedSettingsForm
and store it in the general settings. -
fathershawn β
committed 1a533527 on 1.x authored by
furamag β
Issue #3525416 by furamag, fathershawn, jrockowitz: Add ability to...
-
fathershawn β
committed 1a533527 on 1.x authored by
furamag β