----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34627/#review85092 -----------------------------------------------------------
ambari-web/app/router.js <https://reviews.apache.org/r/34627/#comment136567> A minor nit, but this is getting a little ugly. Perhaps we can simplify this by checking for clusterStatusOnServer once at the top and then doing switch on clusterStatusOnServer.wizardControllerName, case on App.router.get('widgetWizardController.name'), or maybe even better would be to create an object with the mapping and make the routing generic. Like { ..., 'rMHighAvailabilityWizardController': 'main.services.enableRMHighAvailability', 'rollbackHighAvailabilityWizardController': 'main.services.rollbackHighAvailability', ... } and route based on this. Maybe create another JIRA to take care of this in the future. - Yusaku Sako On May 23, 2015, 6:56 a.m., Jaimin Jetly wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34627/ > ----------------------------------------------------------- > > (Updated May 23, 2015, 6:56 a.m.) > > > Review request for Ambari, Srimanth Gunturi, Xi Wang, and Yusaku Sako. > > > Bugs: AMBARI-11359 > https://issues.apache.org/jira/browse/AMBARI-11359 > > > Repository: ambari > > > Description > ------- > > STR: > Navigate through the Create Widget Wizard. > Once it reach metrics and expression page close the browser window > Open another browser window > Expected behavior: This should directly navigate to metrics and expression > page > Actual behavior: opening another browser navigates to dashboard page. When > create/edit widget is launched then it navigated to the metrics and > expression page > > > Diffs > ----- > > ambari-web/app/controllers/main/service/widgets/create/wizard_controller.js > 532ee7a > ambari-web/app/router.js 46e62aa > > Diff: https://reviews.apache.org/r/34627/diff/ > > > Testing > ------- > > Tested the patch to work on live cluster > verified that all unit tests passes successfully > > > Thanks, > > Jaimin Jetly > >
