User photo color dependent on points
Адресира #144
Добавят се параметрите grayscale_user_thumbnails и user_target_points в site.yml:
grayscale_user_thumbnailsпуска feature-а при стойностtrue. По подразбиране стойността еfalse.user_target_pointsотразява очаквания брой точки, които потребител трябва да направи. По подразбиране е 100 и при достигнати толкова точки thumbnail-а на потребителя става напълно цветен.
Първо бях реализирал това изцяло на сървъра, но смятам че css по-удачен начин да се постигне резултата.
Coverage decreased (-0.8%) when pulling 8b4e182927ee660989e8bac35b20ebc131c416f7 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.
Coverage decreased (-0.72%) when pulling 817b7296cfbbcd59846adf486a47fa4b2023303e on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.
Имам притеснения за N+1 query-та, които се получават при показване на всички потребители. Прав ли съм? И има ли как това да се направи по-чисто?
Има ли причина PointsBreakdown да е имплементиран по този начин? Това, което съм направил с model за view-то допустимо ли е?
Има ли причина PointsBreakdown да е имплементиран по този начин?
Кой начин и сравнено с коя алтернатива?
Това, което съм направил с model за view-то допустимо ли е?
Кое точно?
Тестовете минават. Ще трябва да разцъкам приложението за да видя дали нещо се чупи и не е покрито.
Coverage decreased (-0.68%) when pulling bdbb937d1c9cd7d9ced52804672f8866d48d61e7 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.
Coverage decreased (-0.68%) when pulling ef01a1d8dd259a716fbc3c220faf35b3f94d811d on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.
Coverage decreased (-0.68%) when pulling 36185ae391f4bb307dc4451af692002674d0f310 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.
Coverage decreased (-0.75%) when pulling 4fca7fcd5894944c7b6c1d2bd0ec6bd770b05691 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.
@skanev.criticize!