-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename reviews to ratings #4937
Comments
I have already proposed this in the following issue: It will help translators to differentiate between ratings and checkups which both are still called "review". |
This is a different issue. I completely agree with you, but I'm not planning to change the strings/translations yet. |
I've opened up #4984 to change the API urls, splitting that task into a different issue. I'm not sure we'll want to take care of the database yet because it has tons of implications (renaming indices, dealing with the fact that some people might have queries saved in redash, etc) so I am not planning to handle that for now. |
QA: the changes in this issue should be completely transparent to end-users. Please verify that posting/editing/listing/moderating user ratings on add-ons still work. |
Verified on AMO-dev FF57 (Win 10) |
Both internally and in the API, we should rename
reviews
toratings
to avoid clashing withreviewers
(see #4932). For the API we should keep things backwards-compatible for now.The text was updated successfully, but these errors were encountered: