Boris.cirok/190 nefunkcni kontakty

cesko-digital / nasi-politici

Největší otevřená databáze českých politiků a političek. Zjistěte si, kdo vám vládne.

https://www.nasipolitici.cz

MIT License

20 stars 7 forks source link

Boris.cirok/190 nefunkcni kontakty #113

Closed boriscirok closed 3 years ago

boriscirok commented 3 years ago

úprava ESlintu pro hooky
základní refactoring - vyhození duplicit, update API typů, přepis do hooků dotčených komponent
jako důsledek oprava ikonek
posílat request na články s with_text: false, aby zbytečně nechodily texty článků
refactoring detailu politika - vyhození zbytečného kódu, přepis do hooku
refactoring článků
přidání štítků - prozatím pouze "Dezinformační web"

Daidalos117 commented 3 years ago

@boriscirok můžu se zeptat, proč si někde změnil interface za typy? :) Jinak obecně, bylo by fajn jednotlivé tásky dávat do samostatných větví a pull requestů. Tenhle je už docela velký, a obsahuje dost změn. Pokud bychom chtěli aktuálně nasadit opravdu jen opravu ikonek, bude z toho malá komplikace... Nehledě na to, že jsem si ty štítky k dezinfo webů chtěl vzít já, takže jsem z toho teď trochu smutný :( Zkusme se prosím příště nejdříve domluvit (ale to se snad správí až narovnáme to trello)

boriscirok commented 3 years ago

@Daidalos117 Ahoj, jasně, souhlas :) V bodech: interface vs. types) Upřímně tohle je spíš jen ze zkušeností z jiných projektů. Interfaces jsme prakticky nikde nepotřebovali a tady mi to taky nepřišlo nutné. Máš nějakou představu, kde to konkrétně používat jinak? samostané branche) 100 % souhlas, jen jsem měl zrovna čas na tom pracovat a jedna branch vycházela z druhé a neměl jsem tu approve na původní PR. Po nějakém prvotním větším refactoringu kódu je tohle samozřejmost. štítky a rozdělní práce) OK, to mě mrzí, nebyl v tom žádný špatný úmysl, jen jsem vycházel z Trella a komunikace na Signalu. Já bych se do toho taky nějak nehrnul, jen se mi prostě uvolnilo včera a dnes trochu času, tak jsem to chtěl využít právě dle domluvy. Ozvu se na Signalu a probereme to ;)

Daidalos117 commented 3 years ago

Pardon za zpoždění, bohužel se mi stále nedaří zprovoznit lokální backend. Trochu jsem si to namockoval a přijde mi to ok. Za mně jen zase trochu škoda, že měníš syntax komponent ( const componentName -> function componentName() ). Osobně mi stávající styl vyhovuje a tohle mi zase přijde trochu zbytečné, ale už to neřešme :)

boriscirok commented 3 years ago

ad backend) Ono stačí změnit REACT_APP_BASE_API_URL v .env souboru ve složce /Fronted a mělo by ti to jet proti produkci, což by na základní vývoj mělo v pohodě stačit :)

Díky za review.