Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deployment #52

Closed
encukou opened this issue Mar 22, 2016 · 24 comments
Closed

Deployment #52

encukou opened this issue Mar 22, 2016 · 24 comments

Comments

@encukou
Copy link
Member

encukou commented Mar 22, 2016

Tak, už to funguje. Aktuálně to je na http://pyladiescz.readthedocs.org/cs/latest/, což není správná adresa, takže to bude chtít buď přesvědčit ReadTheDocs aby jim s "pyladies.cz" fungovalo "Single version", nebo udělat ten generický deployment na statické weby, co ho má prý @honzajavorek v malíku :)

Honzo, ostrá verze se generuje přes make dirhtml, čímž vznikne statický web v _build/dirhtml.

@SvetlanaM
Copy link
Contributor

môžete tam ešte doplniť a zmergovať zmeny, ktoré som robila dneska? mal by to byt môj posledny commit lebo tu je par veci zle ktore som uz upravila, pripadne ich teda upravim

@encukou
Copy link
Member Author

encukou commented Mar 22, 2016

Tenhle? 4ad092f
Přijde mi že tyhle změny už nejsou potřeba, a z názvu taky není jasné co v ten commit má dělat :(

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

Omlouvám se jestli jsem přepsal nějakou práci, to jsem rozhodně nechtěl :(

@lspdv
Copy link
Member

lspdv commented Mar 23, 2016

Nevim jestli se mam ptat, ale na "stan se PyLady" byval odkaz na aktualni registracni formular pro Prahu. A vsimla jsem si ze Praha materialy maji ruzovou listu tam kde aktualne jsou a Brno ne. Ale mozna je to dotaz tzv. spatny, protoze ja jsem ztratila prehled vcera kolem 12:00

@SvetlanaM
Copy link
Contributor

tam tych zmien bolo vela, tak ja to spojim, ved ked to teraz stiahnem uvidim, stan sa pylady smeruje na uvodnu stranku priamo na tie tri hlavy a tam ho najdu, ci chcete dat do hlavicky obecne iba pre Prahu?

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

Jestli tomu dobře rozumím, hlavička nemá být společná pro všechny stránky?

@SvetlanaM
Copy link
Contributor

ma, to je dobre

@SvetlanaM
Copy link
Contributor

viete tam spojit tento commit 6430e38 ? ci to spravim ja?

@SvetlanaM
Copy link
Contributor

urobim to teda, pozeram ze nie vsetko je uz toho aktualne :)

@honzajavorek
Copy link
Member

@SvetlanaM Já jsem včera koukal na tvoje změny, ale jak je to všechno dokupy a je tam napsané jen "changes", tak jsem se v tom vůbec nevyznal. Popiš to příště trochu líp a rozsekej do jednotlivých změn, ať jde vidět proč ta změna nastala a co v ní je. Spojuje se to pak o dost lépe a nemusela bys to dělat ty.

@encukou Wow, takže hostujeme celý web na RTD? Ha, to mě fakt nenapadlo 😮 Podívám se na to.

@SvetlanaM
Copy link
Contributor

no jasne to uz spravim, som nevedela ze sa to este bude menit, tak som chcela spravit suhrnny update, prave to uz robim :)

@honzajavorek
Copy link
Member

@encukou Můžeš mě přidat do RTD do Owners, ať můžu ladit nastavení?

@SvetlanaM
Copy link
Contributor

oki, mam to done

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

@honzajavorek no, potom co jsem v dokumentaci Sphinxu uviděl Jinju tak mi to nedalo. Jsou tam nějaké problémy, ale myslím že to bude stačit. Minimálně jako staging nebo PR preview by RTD mohlo být fajn, když se trochu ohne.
Jaké máš na RTD jméno? honzajavorek?

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

@SvetlanaM Díky! Jsi skvělá :)

@SvetlanaM
Copy link
Contributor

no ale potrebujem toto https://media.readthedocs.org/css/badge_only.css nejako dostat stade prec :( robi mi to vela problemov v responzivite, konkretne to img co je tam na 100% nastavene a keby to prepisujem u mna to je na dlho, lebo nie kazdy obrazok ma auto alebo 100%. viete to dat prec? alebo nejako dat prioritu pre nase csska?

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

Myslím že tam je problém jen s:

img {
    width: 100%;
    height: auto
}

zbytek vypadá na standardní fa nebo na to mají vlastní třídu.
Myslím že tohle je chyba na jejich straně; ohlásím ji.

Prozatím by se to snad dalo přebít používáním třeba body img {...} v našich CSS, mám to udělat?

@SvetlanaM
Copy link
Contributor

ano iba s tym a ty mas k tomu suboru pristup? nie netreba, musim zatial vymysliet ako to zjednotit, to upravim teda

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

Přístup přímo nemám, ale nahlásil jsem to - readthedocs/sphinx_rtd_theme#144.
Bude-li, čas, můžu se na to pak kouknout do hloubky a poslat jim i pull request...

@SvetlanaM
Copy link
Contributor

@lspdv pokusila som sa to readme doplnit bodmi, skus ci sa ti to podari spustit a je to jasnejsie, ja prave neviem velmi dobre veci vysvetlit a skipnem udaj o ktorom si myslim ze uz ludia vedia. Tak mi daj vediet

@encukou
Copy link
Member Author

encukou commented Mar 23, 2016

@honzajavorek, přidal jsem tě jako maintainera na RTD

@honzajavorek
Copy link
Member

@encukou Díky.

Zkusím nastavit RTD tak, aby fungoval s doménou pyladies.cz. Když to nepůjde, uděláme build přes Travis CI.

Chápu to tedy tak, že starý web pyladies.cz nikomu nebude chybět a já dnes dorazím poslední krok k novému webu, tedy nahození do ostrého provozu, jo? Případné nedostatky (ladění mobilních verzí atd.) bych řešil už pak za provozu.

@honzajavorek
Copy link
Member

@encukou Vypadá to, že to možná bude nějak fungovat, ale (samozřejmě, jako obvykle) nebude fungovat apex doména. To znamená, že pyladies.cz fungovat nebude, ale www.pyladies.cz ano.

Řešení asi jedině přes ten CloudFront, jak to kdysi radil @VerosK.

@honzajavorek
Copy link
Member

screen shot 2016-03-23 at 21 53 36

Díky všem za spolupráci! 🎉 🎺 😸

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants