text: Title: Extra ) in apps.py?
  Body: ### Issue Summary
  
  Is there an extra parenthesis [here](https://github.com/django-oscar/django-oscar/blob/master/src/oscar/apps/catalogue/reviews/apps.py#L26)?
  
  Should
  ```
   path('<int:pk>)/vote/', login_required(self.vote_view.as_view()), name='reviews-vote'),
  ```
  be
  ```
   path('<int:pk>/vote/', login_required(self.vote_view.as_view()), name='reviews-vote'),
  ```
  ?