Add a general comments feature class

Review Request #8471 — Created Oct. 17, 2016 and submitted

Information

Review Board
release-3.0.x

Reviewers

The new general comments feature can be enabled or disabled with the
reviews.general_comments feature. Disabling this feature will disable
all UI elements for it, as well as the web API resources for it.

Tested with reviews.general_comments enabled and disabled. When
disabled, general comments cannot be created and the UI elements do not
appear.

Description From Last Updated

local variable 'general_comments_enabled' is assigned to but never used

reviewbotreviewbot

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

Classes should go before instances.

chipx86chipx86

More specifically, it allows comments to be made. Issues are optional.

chipx86chipx86

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot

This is invoking this._onAddCommentClicked immediately.

chipx86chipx86

'django_reset' imported but unused

reviewbotreviewbot

'from settings_local import *' used; unable to detect undefined names

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/reviews/views.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/reviews/views.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
  2. reviewboard/reviews/views.py (Diff revision 1)
     
     
    Show all issues
     local variable 'general_comments_enabled' is assigned to but never used
    
  3. reviewboard/settings.py (Diff revision 1)
     
     
    Show all issues
     'django_reset' imported but unused
    
  4. reviewboard/settings.py (Diff revision 1)
     
     
    Show all issues
     'from settings_local import *' used; unable to detect undefined names
    
  5. 
      
brennie
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
    
    Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
  2. reviewboard/settings.py (Diff revision 2)
     
     
    Show all issues
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 2)
     
     
    Show all issues
     'from settings_local import *' used; unable to detect undefined names
    
  4. 
      
brennie
brennie
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
  2. reviewboard/settings.py (Diff revision 3)
     
     
    Show all issues
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 3)
     
     
    Show all issues
     'from settings_local import *' used; unable to detect undefined names
    
  4. 
      
brennie
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
  2. reviewboard/settings.py (Diff revision 4)
     
     
    Show all issues
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 4)
     
     
    Show all issues
     'from settings_local import *' used; unable to detect undefined names
    
  4. 
      
brennie
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
  2. reviewboard/settings.py (Diff revision 5)
     
     
    Show all issues
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 5)
     
     
    Show all issues
     'from settings_local import *' used; unable to detect undefined names
    
  4. 
      
david
  1. Ship It!
  2. 
      
chipx86
  1. 
      
  2. reviewboard/reviews/features.py (Diff revision 5)
     
     
     
     
     
    Show all issues

    Classes should go before instances.

  3. reviewboard/reviews/features.py (Diff revision 5)
     
     
    Show all issues

    More specifically, it allows comments to be made. Issues are optional.

  4. Show all issues

    This is invoking this._onAddCommentClicked immediately.

  5. 
      
brennie
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/reviews/features.py
        reviewboard/settings.py
        reviewboard/reviews/default_actions.py
        reviewboard/webapi/resources/base_review_general_comment.py
    
    Ignored Files:
        reviewboard/static/rb/js/views/reviewDialogView.js
        reviewboard/static/rb/js/pages/views/reviewablePageView.es6.js
        reviewboard/templates/reviews/reviewable_page_data.js
    
    
  2. reviewboard/settings.py (Diff revision 6)
     
     
    Show all issues
     'django_reset' imported but unused
    
  3. reviewboard/settings.py (Diff revision 6)
     
     
    Show all issues
     'from settings_local import *' used; unable to detect undefined names
    
  4. 
      
brennie
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.0.x (1fc7458)