We fevo #103
We fevo #103
Conversation
|
Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
|
|
|
| const hasFevo = document.querySelector('a.fevo-btn'); | ||
| if (hasFevo) { | ||
| injectFevoScript(); | ||
| } | ||
|
|
||
| const hasWeFevo = document.querySelector('a.we-fevo-btn'); | ||
| if (hasWeFevo) { |
There was a problem hiding this comment.
if (hasFevo || hasWeFevo)?
There was a problem hiding this comment.
I don't think so?
Not at all confident this is right, but my understanding from Gary is that there are 2 types of fevo buttons, but they aren't related beyond the fact that they come from the same vendor. but also that there could at least in theory exist both types of buttons on the same page.
so they presence of a "normal" fevo button does not necessitate loading the sdk for the "we fevo" type, and vice versa.
There was a problem hiding this comment.
Wow. I definitely read this wrong. Thank you.
|
https://wefevo--theplayers--hlxsites.hlx.page/hospitality
extend fevo button decoration to support weFevo.