Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

updated the explanation and added some discussion points

  • Loading branch information...
commit cc1a496dd89c40975f5811c70a27099b5cde516e 1 parent 250ec85
@paulcc authored
Showing with 15 additions and 2 deletions.
  1. +15 −2 README.markdown
View
17 README.markdown
@@ -1,7 +1,7 @@
Reviews
=======
-Straightforward review facility
+Straightforward review/rating facility
Please see TODO file
@@ -18,7 +18,7 @@ Three partials:
- ./app/views/products/_shortrating.html.erb -- shorter version of above
- ./app/views/products/_review.html.erb -- display a single review
-Admin interface yet to be done!
+Administrator can edit and/or approve and/or delete reviews.
Implementation
@@ -28,3 +28,16 @@ reviews table is quite obvious - and note the "approved" flag which is for the
administrator to update
ratings table holds current fractional value - avoids frequent recalc...
+
+
+Discussion
+----------
+
+Some points which might need modification in future:
+ - I don't track the actual user on a review (just their "screen name" at the
+ time), but we may want to use this information to avoid duplicate reviews
+ etc.
+
+ - Rating votes are tied to a review, to avoid spam. However: ratings are
+ accepted whether or not the review is accepted. Perhaps they should only
+ be counted when the review is approved.
Please sign in to comment.
Something went wrong with that request. Please try again.