search

gugcz / web-frontend

Frontend for new GUG.cz web
https://gug.cz
4 stars 0 forks source link

Partners section and footer update #32

Closed bartusek27 closed 4 years ago

bartusek27 commented 4 years ago

Teď koukám, že jsem uploadnul logo, přitom ho tahám z netu...

bartusek27 commented 4 years ago

Tak obrázek smazán. Může někdo schválit?

HormCodes commented 4 years ago

@bartusek27 Díky moc za pull request! Sám jsem teď spíše v neaktivním módu, ale snad si mohu dovolit malý komentář:

Bylo by fajn dát na Slack nějaký post ohledně nového partnera. Pokoušel jsem se sám dohledat, nic jsem nenašel a myslím, že nejsem sám, kterého zajímá jak je partnerství nastavené.

Layout je trochu rozhozený, takže bude potřeba ještě trochu opravit a při dané opravě bych byl raději pro variantu s obrázkem (SVG) v repozitáři, protože link se může relativně rychle změnit. Nicméně tak jako tak loga načítána nebudou protože je nutný větší zásah do sekce "Organizátoři". Rád bych to vyřešil, ale bohužel aktuální priority mi to nedovolují.

Ještě jednou, moc velké díky za pull request.

bartusek27 commented 4 years ago

@HormCodes Ahoj, v pohodě - co se týká partnera - je to zatím ve fázi přípravy, toto bylo domluveno s Filipem a jedná se o aplikaci na unifikování autorizace pod jednu entitu, která není Slack - pomalu přecházíme na G Suite autorizaci, ale chceme nechat možnost přihlásit se přes Slack - proto jsem domluvil licenci zdarma u Auth0. Více informací bude následovat. Zatím tím nechci nikomu plést hlavu a vše ve správnou chvíli zveřejníme. Pokud tě zajímá více, máme na Slacku tajný kanál #digirevoluce 😆

S layoutem jsem na lokálu neměl problém, ale na produkci jsem si toho všiml, takže jsem již ty obrázky trochu upravil v #33 . S tím lokálním obrázkem souhlas, ale když si všimneš, tím že se nenačítá ze serveru tak se "vůbec načte"... Navíc jakýkoliv request na cizí servery nám šetří peníze, ne? Ohledně lazy-loadingu - očividně se to přidáním moc nevyřešilo, ty obrázky orgů se asi chovají tak, že jsou zobrazeny všechny, což je škoda a asi to opravdu bude chtít větší úpravy, na to ale už nemám.

Díky za reakci!