Comment 4 for bug 1993824

Revision history for this message
Bill Erickson (berick) wrote :

Did a quick pass. Looking really good to me.

In addition Jane's comments:

* The progress dialog would be much more useful if it reported progress instead of being indeterminate.
* There's a TODO to replace a window.alert().
* Love the use of $localize.. I'm curious if it works with string concat vs. templated `here is a ${string} that translates`
* Some of the console.logs have TODOs to be replaced with toasts.
* If you create a session without verifying up front, is there a way to verify the entire batch instead of going page by page?
* Would be handy if the Redirected To column showed or linked to something useful.