[rbxmlreview] Modernize the XML Review extension.

Review Request #11081 — Created July 15, 2020 and submitted

Information

rb-extension-pack
master

Reviewers

The XML review UI extension was pretty long in the tooth, and wasn't
using static media or the review UI JavaScript infrastructure in the
correct way. This change updates it so that it doesn't need its own
template, and relies on static media bundles instead of manually loading
the JavaScript file.

Enabled the extension. Uploaded an XML file and was able to review it.

Summary ID
[rbxmlreview] Modernize the XML Review extension.
The XML review UI extension was pretty long in the tooth, and wasn't using static media or the review UI JavaScript infrastructure in the correct way. This change updates it so that it doesn't need its own template, and relies on static media bundles instead of manually loading the JavaScript file. Testing Done: Enabled the extension. Uploaded an XML file and was able to review it.
35ccc0f28f6ae38905e9f5c07a0e93b2cfb10744
Description From Last Updated

"Testing Done" didn't make it into the right field.

chipx86chipx86

E999 SyntaxError: invalid syntax

reviewbotreviewbot

Bare imports go before from ... imports.

chipx86chipx86

You can just access XMLReviewUIExtension.instance.

chipx86chipx86

F401 'rbxmlreview.extension.XMLReviewUIExtenison' imported but unused

reviewbotreviewbot

F821 undefined name 'XMLReviewUIExtension'

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

flake8

david
chipx86
  1. 
      
  2. Show all issues

    "Testing Done" didn't make it into the right field.

  3. rbxmlreview/rbxmlreview/reviewui.py (Diff revision 2)
     
     
     
     
     
     
    Show all issues

    Bare imports go before from ... imports.

  4. rbxmlreview/rbxmlreview/reviewui.py (Diff revision 2)
     
     
     
    Show all issues

    You can just access XMLReviewUIExtension.instance.

  5. 
      
david
Review request changed
Description:
   

The XML review UI extension was pretty long in the tooth, and wasn't

    using static media or the review UI JavaScript infrastructure in the
    correct way. This change updates it so that it doesn't need its own
    template, and relies on static media bundles instead of manually loading
    the JavaScript file.

-  
-  

Testing Done:

-   Enabled the extension. Uploaded an XML file and was able to review it.

Testing Done:
  +

Enabled the extension. Uploaded an XML file and was able to review it.

Commits:
Summary ID
[rbxmlreview] Modernize the XML Review extension.
The XML review UI extension was pretty long in the tooth, and wasn't using static media or the review UI JavaScript infrastructure in the correct way. This change updates it so that it doesn't need its own template, and relies on static media bundles instead of manually loading the JavaScript file. Testing Done: Enabled the extension. Uploaded an XML file and was able to review it.
80cbe87c06c8d8ee333ccf0de267e6caac7c0065
[rbxmlreview] Modernize the XML Review extension.
The XML review UI extension was pretty long in the tooth, and wasn't using static media or the review UI JavaScript infrastructure in the correct way. This change updates it so that it doesn't need its own template, and relies on static media bundles instead of manually loading the JavaScript file. Testing Done: Enabled the extension. Uploaded an XML file and was able to review it.
66aabb45e617f21e2ce5ba789b3a8f1bf607be0d

Checks run (1 failed, 1 succeeded)

flake8 failed.
JSHint passed.

flake8

david
chipx86
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to master (5a5705b)