search

kir-dev / korok

Villanykari Profil és Körök alkalmazás
MIT License
6 stars 0 forks source link

Landing page #119

Closed Kresshy closed 10 years ago

Kresshy commented 10 years ago

The landing page is finished. The build and the necessary controllers are implemented. The authentication module also implemented in this PR. At review please focus on the auth module and on the landing page!

vbalazs commented 10 years ago

az auth.js-t nagyon at kene nezni vagy megvizsgalni, hogy miert nem hasznalhato 3rd party (unit tesztelt) lib

tmichel commented 10 years ago

Szerintem nagyjából jó lesz. A .gitignore-t okosítani kell. Lehet elég egy a front-end mappába, nem kell a globális ignore-ba tenni mindent.

Az autentikációs résznél lehet, hogy érdemes lenne valami libet használni, vagy ha már sajátot írunk, akkor emeljük ki egy saját modulba, ami teljesen független az alkalmazástól. Ha jól sikerül, akkor akár npm package-ként is lehet publikálni a világnak.

Egy-két kritikus részhez lehet érdemes lenne teszteket írni. Növelné a biztonság érzetem, no meg jó példát állítana az eljövendő fejlesztők elé.

tmichel commented 10 years ago

menjen a merge! aztán majd lesz valami.