[review-together] Modernize the review-together codebase.

Review Request #11087 — Created July 20, 2020 and submitted

david
rb-extension-pack
master
rb-extension-pack

This change updates the review-together extension, porting the JS side
to ES6, updating the Python code for Django 1.11/Python 3/Review Board
4.0 compatibility, and converting the CSS to use our new color
definitions.

The visual appearance isn't super polished (many of the colors are a
little bit clashy with TogetherJS's icons), but I haven't changed
anything there. Updating to the latest TogetherJS and polishing the UI
might make for a good student project at some point.

Installed and activated the extension. Verified that the settings form
and frontend UI worked as expected.

Summary
[review-together] Modernize the review-together codebase.
Description From Last Updated

Minor thing, but while updating the code, we should probably just add docs for this.

chipx86chipx86

Col: 17 Missing 'new' prefix when invoking a constructor.

reviewbotreviewbot

Col: 17 Missing 'new' prefix when invoking a constructor.

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 passed.
JSHint failed.

JSHint

chipx86
  1. 
      
  2. Minor thing, but while updating the code, we should probably just add docs for this.

  3. 
      
david
Review request changed

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint

chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to master (359d6d2)
Loading...