evans icon indicating copy to clipboard operation
evans copied to clipboard

User photo color dependent on points

Open sgatev opened this issue 11 years ago • 11 comments

Адресира #144

Добавят се параметрите grayscale_user_thumbnails и user_target_points в site.yml:

  • grayscale_user_thumbnails пуска feature-а при стойност true. По подразбиране стойността е false.
  • user_target_points отразява очаквания брой точки, които потребител трябва да направи. По подразбиране е 100 и при достигнати толкова точки thumbnail-а на потребителя става напълно цветен.

Първо бях реализирал това изцяло на сървъра, но смятам че css по-удачен начин да се постигне резултата.

sgatev avatar Sep 20 '14 18:09 sgatev

Coverage Status

Coverage decreased (-0.8%) when pulling 8b4e182927ee660989e8bac35b20ebc131c416f7 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.

coveralls avatar Sep 20 '14 18:09 coveralls

Coverage Status

Coverage decreased (-0.72%) when pulling 817b7296cfbbcd59846adf486a47fa4b2023303e on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.

coveralls avatar Sep 23 '14 15:09 coveralls

Имам притеснения за N+1 query-та, които се получават при показване на всички потребители. Прав ли съм? И има ли как това да се направи по-чисто?

skanev avatar Sep 23 '14 20:09 skanev

Има ли причина PointsBreakdown да е имплементиран по този начин? Това, което съм направил с model за view-то допустимо ли е?

sgatev avatar Sep 24 '14 09:09 sgatev

Има ли причина PointsBreakdown да е имплементиран по този начин?

Кой начин и сравнено с коя алтернатива?

Това, което съм направил с model за view-то допустимо ли е?

Кое точно?

skanev avatar Sep 24 '14 09:09 skanev

Тестовете минават. Ще трябва да разцъкам приложението за да видя дали нещо се чупи и не е покрито.

sgatev avatar Sep 24 '14 09:09 sgatev

Coverage Status

Coverage decreased (-0.68%) when pulling bdbb937d1c9cd7d9ced52804672f8866d48d61e7 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.

coveralls avatar Sep 24 '14 09:09 coveralls

Coverage Status

Coverage decreased (-0.68%) when pulling ef01a1d8dd259a716fbc3c220faf35b3f94d811d on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.

coveralls avatar Sep 24 '14 10:09 coveralls

Coverage Status

Coverage decreased (-0.68%) when pulling 36185ae391f4bb307dc4451af692002674d0f310 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.

coveralls avatar Sep 24 '14 11:09 coveralls

Coverage Status

Coverage decreased (-0.75%) when pulling 4fca7fcd5894944c7b6c1d2bd0ec6bd770b05691 on s2gatev:user-photo-color-dependent-on-points into 7d363210cd2117f50e60f82562e2e3addf1f79b8 on skanev:master.

coveralls avatar Sep 24 '14 22:09 coveralls

@skanev.criticize!

sgatev avatar Oct 01 '14 06:10 sgatev