Closed martijncasteel closed 4 years ago
Biggest problem is the sidebar, however I used bootstrap 3 so It doesn't look to bad without flatify. Person who would do this could also look at bootstrap 4, but this has nothing to do with Rails anymore pure (s)css here. And some pages are horrible such as home. I say doable, if someone is creative. cc @tomwassenberg @arianvp
For koala we can build a minimalistic css file just putting some things in the correct place. For Sticky we can overwrite this file with our specific flatify style, perhaps this would be possible using a git patch we can execute everytime some changes have been made to the public repo.
git apply add-flatify.patch
Patching entire features would not be desirable because it is just to complex to debug or prone to upcoming changes. CSS files are easy to overwrite, html or code is not.
We will have to Cherry pick The commits of flatify out of The Commit history and force push as well On 26 Apr 2016 14:02, "Martijn Casteel" notifications@github.com wrote:
For koala we can build a minimalistic css file just putting some things in the correct place. For Sticky we can overwrite this file with our specific flatify style, perhaps this would be possible using a git patch we can execute everytime some changes have been made to the public repo.
git apply add-flatify.patch
— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub https://github.com/svsticky/constipated-koala/issues/181#issuecomment-214717877
True we have to remove it's entire existence, But I think this is a doable option, storing the patch file within skyblue
for copying. The patch should include the require in the application.scss and include the additional file flatify.css
. Purging the file would do https://help.github.com/articles/remove-sensitive-data/
@svsticky/it-crowd except for the stylesheet this project can be open-sourced right?
I think so, but like @arianvp said we shouldn't forget all metadata associated with the stylesheet, like commit messages and certain issues like this ;)
We can still have the license for this stylesheet, but where not licensed for sharing it.
Dus, zoals ik er nu naar kijk heeft koala teveel maatwerk om te open-sourcen, we geven daarmee teveel informatie weg van de huidige checkout.
Onzin het is GPL code
Het open-sourcen van o.a. Koala zal worden opgenomen in het beleidsplan voor volgend jaar. Ik vind het een belangrijk goed om als studievereniging bij te dragen aan de open-source community. Graag hoor ik van jullie waar ik op moet letten (en waar aan gewerkt moet worden) om dit voor Koala mogelijk te maken.
Beleidsplan, heftig. Maar stylesheet flatify is het enige wat licensed is, verder moet de upgrade nog doorgaan zodat er security patches zijn gedaan en ik ben bezig de stylesheets iets makkelijker op te zetten, daarna wordt het vrij makkelijk, ik zou het wel leuk vinden als het template gebruikt (misschien met een patch bij uitrollen). Heb je al eens geprobeerd flatify.css eruit te halen, werkt nog wel een beetje.
Verder moeten er dingen vertaald worden als je ook engels wil supporten @spasticVerbalizer. Wat ik wel nadelig vind, het is nooit out-of-the-box, dat wij veel dingen gebruiken zoals mailgun, mollie en amazon waar andere niet meteen gebruik van maken.
@emielvanrijn
Eigenlijk zou je ook goede tests willen schrijven, je wil niet dat dingen stuk gaan. Laat staan onveilig worden. gemnasium is wel een leuke github plugin die je dan kan gebruiken.
Het voordeel van open-sourcen is wel dat als mensen zouden zeuren over iets dat komt door (een gebrek aan) tests, je kan zeggen dat PR's altijd welkom zijn ;)
@emielvanrijn misschien ook even alle niet gebruikte branches verwijderen. En kijken of mensen zich kunnen houden aan de contribution formatting. In de open-source community worden dingen daar op afgewezen. https://github.com/svsticky/constipated-koala/blob/development/CONTRIBUTING.md
Branch naming, Try to be descriptive. Use the following prefixes for the names depending on the type of work:
feature/ for new features.
bug/ for bugfixes.
doc/ for documentation.
test/ for testing.
debt/ for refactoring and enhancements.
We moederen ook even overwegen of we de hele history willen publiceren of dat van een nieuwe "genesis" willen doen
Moeten* (sorry, kan niet editen op mobiel)
Ik denk dat dat wel kan, dan kan iemand ook een oudere tag gebruiken. Maar we moeten wel flatify overal purgen
Ik bedacht me net dat alle tags ook weggegooid moeten worden.
Denk dat het makkelijkste is om de Git history volledig weg te gooien voor de public release
For now, we will leave flatify in the repo. Although not completely legal, we think it is not worth the hassle now. A good time for recreating or purging the repo would be the summer holidays, when no active development is going on. I will make a new issue, referring to this one
Constipated-koala
is started with the idea of open-sourcing it. However along the way I decided to use Flatify (because it was hidious!) which is a styling under a license, hence we can not share the styling.If ever someone would want to change the style it should be done by replacing flatify.css, than we can open-source it and everyone can help or use the api.
Last few months I been realy working on documentation and I think for now it will suffice, if you want to you can run it. (nginx config might be to specific for sticky)