search

pavelevap / ceske-sluzby

České služby pro WordPress
56 stars 25 forks source link

Trocha WC 3.0.0 kompatibility #196

Open vyskoczilova opened 6 years ago

vyskoczilova commented 6 years ago

Kompatibilita s WC 3.0.0 (ošetřeno podmínkou):

Fixy:

pavelevap commented 5 years ago

Díky moc za PR, konečně se po roce dostávám k další práci na pluginu! :tada:

Přijde mi to ale nějaké porouchané (nebo už je hodně večer :-)). Když se podívám na přehled všech změn (nikoli jednotlivé commity), tak se úpravy porovnávají s nějakou starší verzí místního repozitáře (i když to tak nevypadá). Schválně přidám nějaké komentáře...

vyskoczilova commented 5 years ago

Gratuluji, to je super zpráva! :1st_place_medal:

Hm, to je divný, myslela jsem, že jsem si to zaktualizovala z masteru. Máš nějaký postup, jak to dát do pořádku?

pavelevap commented 5 years ago

Moc nevím jak to nejlépe opravit, myslím, že se to rozbilo tímto commitem: https://github.com/pavelevap/ceske-sluzby/commit/6f5b7b253e02106e97ca83e9a4a9823e318fc350

Zkusit to celé nějak aktualizovat, nakonec zbude asi jen pár commitů, ideálně pro každou issue samostatný PR. Jestli na to nemáš čas, tak to z toho klidně "vytahám" a uvedu tě tam jako autorku a rovnou to doplním do masteru. Některé objevené chybky jsou na :man_facepalming: :-)

vyskoczilova commented 5 years ago

Ahoj, smazala jsem nakonec svůj fork (tady to zůstalo), zkusím to ještě jednou projít a poslat to postupně jako PR. Zvládneš to nějak postupně odklepat? Budu potřebovat tvůj plugin u několika klientů upravit o nějaké drobnosti a nerada bych to fixovala do neopravené verze.

vyskoczilova commented 5 years ago

Tak jsem to prošla a vypadá to, že vůči aktuální verzi jsou tam jenom ty dva PR, tohle pak můžeme zavřít. Máš tu nějakej zásadní blocker před releasem verze?