search

redbitcz / simpleshop-wp-plugin

SimpleShop Wordpress Plugin
https://www.simpleshop.cz/category/wordpress-plugin/
Other
2 stars 1 forks source link

Remove `lc` hack #48

Closed jakubboucek closed 4 years ago

Dosty85 commented 4 years ago

1) Koukám, že se tu generuje stále starý JS kód. Nedávalo by smysl tam dávat nový? Přichází z API v produktech. 2) Tím by se vyřešila i část, kdy by se to dalo testovat na LC.

jakubboucek commented 4 years ago
  1. Koukám, že se tu generuje stále starý JS kód. Nedávalo by smysl tam dávat nový? Přichází z API v produktech.

Nerozumím posledně větě. Gutenberg generuje nový, téhle je ještě takhle. Nevím popravdě, jaké má nový výhody. Klidně můžeme předělat.

  1. Tím by se vyřešila i část, kdy by se to dalo testovat na LC.

Jak to ten nový vyřeší, když se bude tahat ze serveru?

Dosty85 commented 4 years ago
  1. Koukám, že se tu generuje stále starý JS kód. Nedávalo by smysl tam dávat nový? Přichází z API v produktech.

Nerozumím posledně větě. Gutenberg generuje nový, téhle je ještě takhle. Nevím popravdě, jaké má nový výhody. Klidně můžeme předělat.

Starý kód bych už nikde negeneroval. Nový má výhodu například v předávání GA atp.

  1. Tím by se vyřešila i část, kdy by se to dalo testovat na LC.

Jak to ten nový vyřeší, když se bude tahat ze serveru?

Protože když to budeš testovat na locale, tak to budeš testovat vůči lokálnímu SS a ten ti vrátí kód s czlc.

jakubboucek commented 4 years ago

Pro úplnost: Doplnil jsem do debug módu možnost nastavit vlastní URL formuláře. Tíme je vyřešena otázka testování.