131 show dynamic score - Githubissues

AI-comp / wpcs

2 stars 2 forks source link

131 show dynamic score #197

Closed Genki-S closed 11 years ago

Genki-S commented 11 years ago

Not real time. Because 1) It would be not smart to have score calculation formula in two place, i.e. in ruby code and in js code 2) I think Ajax is too much

I think this suffice for the purpose. Close #131.

p.s. I appreciate any suggestion to make this real time. I'm out of ideas...

murooka commented 11 years ago

This commit will collide if it would be merged. Please rebase the master.

coveralls commented 11 years ago

Coverage decreased (-0%) when pulling adf4edb8f81c846adf85454ab27b08b714d3382d on 131_show_dynamic_score into 8df209eec3b001e8f403110c12508f0fd317fa6a on master.

Genki-S commented 11 years ago

I am sorry for sluggish work. I was looking for the best way to rebase. I was surprised that some resources say it is OK to rebase and do force push because topic branch is temporary thing.

This saved me from a labor of deleting this branch and opening pull request again. But if I am missing something, please let me know.