Fix more errors in Review Bot configuration pages.

Review Request #11194 — Created Sept. 21, 2020 and submitted

Information

ReviewBot
release-2.0.x

Reviewers

This makes two changes to fix additional errors in the Review Bot config
process.

First, the integration config form was still encountering issues with
client-side validation of hidden tool options input elements on some
versions of Chrome. Instead of going through and making each element
disabled, I've changed it to just set novalidate on the top-level
form.

Second, I noticed that there was an error in the JS console when loading
the extension configuration page. This was due to the template
overriding the scripts-post block but not including the block's super,
leaving out some necessary JS files.

Exercised the extension and integration config pages.

Summary ID
Fix more errors in Review Bot configuration pages.
This makes two changes to fix additional errors in the Review Bot config process. First, the integration config form was still encountering issues with client-side validation of hidden tool options input elements on some versions of Chrome. Instead of going through and making each element disabled, I've changed it to just set `novalidate` on the top-level form. Second, I noticed that there was an error in the JS console when loading the extension configuration page. This was due to the template overriding the `scripts-post` block but not including the block's super, leaving out some necessary JS files. Testing Done: Exercised the extension and integration config pages.
e81c40e1315dc2878d13c9235f35298f42cfa5de
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (67fbfe6)
Loading...