----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/43365/#review118959 -----------------------------------------------------------
ambari-web/app/mixins/wizard/assign_master_components.js (line 1089) <https://reviews.apache.org/r/43365/#comment180268> Does this line below require ```self.set('submitButtonClicked', false);``` ? - jun aoki On Feb. 11, 2016, 1:02 a.m., Matt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/43365/ > ----------------------------------------------------------- > > (Updated Feb. 11, 2016, 1:02 a.m.) > > > Review request for Ambari, Alexander Denissov, Alejandro Fernandez, Aleksandr > Kovalenko, Alexandr Antonenko, bhuvnesh chaudhary, Goutam Tadi, Jaimin Jetly, > jun aoki, Lav Jain, Newton Alex, Oleksandr Diachenko, and Sumit Mohanty. > > > Bugs: AMBARI-14701 > https://issues.apache.org/jira/browse/AMBARI-14701 > > > Repository: ambari > > > Description > ------- > > There was noAction onSecondary (Cancel button). Implemented onSecondary, and > modified the callback for the same. > > > Diffs > ----- > > ambari-web/app/mixins/wizard/assign_master_components.js 7dc267e > > Diff: https://reviews.apache.org/r/43365/diff/ > > > Testing > ------- > > All unit tests passed: > 10384 tests complete (9 seconds) > 121 tests pending > > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 58.283 s > [INFO] Finished at: 2016-02-10T16:55:55-08:00 > [INFO] Final Memory: 12M/311M > [INFO] ----------------------------------------------------------------------- > > > Thanks, > > Matt > >
