search

lostm1nd / HackBulgaria

Projects related to the courses at HackBulgaria
1 stars 0 forks source link

Team Matcher App - SCREAMING_SNAKE_CASE за модулите #14

Closed RadoRado closed 10 years ago

RadoRado commented 10 years ago

Идеята да разделиш нещата на модули е много добра.

Имам само една забележка за имената им - този SCREAMING_SNAKE_CASE се ползва в Python, докато в JavaScript ModuleName би било предостатъчно :+1:

lostm1nd commented 10 years ago

Можеш ли да погледнеш сега модулите, защото исках да ги заредя с RequireJS, но нещо се чупи във филтриращия модул. Подал съм му зависимост към display модула, но не ми го зарежда и ми седи undefined.

В app.js няма такива проблеми. Там зарежда модулите и показва първоначалната страница, но после, когато искам да ги филтрирам и влезна във филтриращия модул там displayModule е undefined и не мога да скрия филтрираните студенти.

RadoRado commented 10 years ago

Circular dependencies беше проблема ;) Важното е, че го хванахме!