• 
      

    Add a site setting for the static URL

    Review Request #7979 — Created Feb. 19, 2016 and submitted

    Information

    Review Board
    release-2.0.x

    Reviewers

    This patch adds a field to the general site settings form for
    configuring the URL of served static media.

    Updated the field and verified requests were made to the new URL.

    Description From Last Updated

    Can you include instructions about leaving this blank similar to site_media_url?

    daviddavid

    Let's clarify just a bit more, saying: "to the static files, such as the JavaScript, CSS, and image files bundled …

    chipx86chipx86
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/admin/forms.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/admin/forms.py
      
      
    2. 
        
    david
    1. What's the use case here?

      1. Configuring usage of a CDN without having to go through management commands. We already have this for uploaded media. I've hit a need for this so many times with RBCommons (when CloudFront fell over once, and often during deployments when the CDN hasn't yet updated and we need to temporarily revert back to /static/).

    2. 
        
    david
    1. typo in description: ot -> to

    2. reviewboard/admin/forms.py (Diff revision 1)
       
       
      Show all issues

      Can you include instructions about leaving this blank similar to site_media_url?

    3. 
        
    brennie
    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/admin/forms.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/admin/forms.py
      
      
    2. 
        
    david
    1. Ship It!
    2. 
        
    chipx86
    1. 
        
    2. reviewboard/admin/forms.py (Diff revision 2)
       
       
      Show all issues

      Let's clarify just a bit more, saying: "to the static files, such as the JavaScript, CSS, and image files bundled with Review Board or third-party extensions." That way it's slightly more clear what people will be dealing with here.

    3. 
        
    brennie
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.0.x (b3dfbed)