- Issue created by @mherchel
- First commit to issue fork.
- last update
over 1 year ago 29,873 pass, 7 fail - @casseywaldo opened merge request.
- 🇺🇸United States CasseyWaldo
This is a collaboration between kwiseman, bernardm28, and CasseyWaldo.
- 🇺🇸United States kwiseman
We noticed that Umami has the file: /core/profiles/demo_umami/config/install/core.entity_view_mode.node.card.yml
Does it need to be removed to avoid a conflict?
- 🇺🇸United States bernardm28 Tennessee
We also noticed teaser is included in a lot of twig comment blocks. Which if this is included should those also reference card as an example?
Maybe it good a good opportunity to update some of those comments as those tend to provide people with options. - last update
over 1 year ago 29,906 pass, 7 fail - last update
over 1 year ago 29,906 pass, 7 fail - 🇬🇧United Kingdom catch
@kwiseman there should be zero conflict, the Umami theme and Umami profile are mutually exclusive with standard/olivero/claro. The only time it comes up is if you install umami and switch theme.
- Status changed to Needs review
over 1 year ago 6:35pm 3 August 2023 - Status changed to Needs work
over 1 year ago 6:58pm 3 August 2023 4:35 1:20 Running- @bernardm28 opened merge request.
- 🇷🇺Russia kostyashupenko Omsk
Actually, is there a reason for v2 merge request @bernardm28?
- 🇺🇸United States bernardm28 Tennessee
Not other than trying to figure out why it broke some tests.
Great feedback. I will add some notes in gitlab for most things it could probably be tweak.
That said, I'm not sure why it would still fail. Do you know? what trigger the test to fail? @kostyashupenko