search

drupalhu / drupal.hu

drupal.hu code base
9 stars 12 forks source link

Issue #219 - Rename text formats which have numeric machine-name #229

Closed Sweetchuck closed 5 years ago

Sweetchuck commented 9 years ago

Issue #219 - Rename text formats which have numeric machine-name

Sweetchuck commented 9 years ago

Javasolnám, hogy ez PR egymagában menjen ki az élesre, mert más módosításokkal össze akadhat a cache ürítések miatt.

tanarurkerem commented 9 years ago

Szia Andor!

Hogyan tesztelted? Hogyan lehet tesztelni ezt?

Sweetchuck commented 9 years ago

Úgy hogy átnéztem az összes űrlapot ahol a szöveg formátumok használatban vannak vagy konfigurációba vannak mentve.

tanarurkerem commented 9 years ago

Azt nézem, hogy a view-k-ban ami a db-ben van azoknál updateled (ha jól értem) de a featrue-okban lévő view-kat nem. Vagy azokat nem is kell, vagy nincs is, vagy mi az elképzelés?

pp

Sweetchuck commented 9 years ago

Ez most miért kérdezed?

Vagy azokat nem is kell, vagy nincs is, vagy mi az elképzelés?

Nem az lenne a lényege a review-nak, hogy átnézd hogy mindent jól csináltam-e? Aztán szólni, hogy ezt meg ezt kihagytad

tanarurkerem commented 9 years ago

Hogyan döntsem el, hogy jól csináltad-e, ha nem tudom, hogy mit csináltál (mert nem minden látszik a kódból ugye), és amit látok azt se tudom, hogy miért és miért úgy csináltál.

pp