----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57505/#review168600 -----------------------------------------------------------
Ship it! Ship It! - DIPAYAN BHOWMICK On March 10, 2017, 11:44 a.m., Madhan Reddy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57505/ > ----------------------------------------------------------- > > (Updated March 10, 2017, 11:44 a.m.) > > > Review request for Ambari, belliraj hb, DIPAYAN BHOWMICK, Gaurav Nagar, and > Pallav Kulshreshtha. > > > Bugs: AMBARI-20360 > https://issues.apache.org/jira/browse/AMBARI-20360 > > > Repository: ambari > > > Description > ------- > > Added error message to show if workflow xml doesn't contain workflow-app tag, > similarly coordinator-app tag for coordinator xml and bundle-app tag for > bundle xml. And also added the check for maximum number of nodes to render > and shown the error message if it exceeds maximum. > > > Diffs > ----- > > > contrib/views/wfmanager/src/main/resources/ui/app/components/bundle-config.js > e94d51a > > contrib/views/wfmanager/src/main/resources/ui/app/components/coord-config.js > c87ea99 > > contrib/views/wfmanager/src/main/resources/ui/app/components/designer-errors.js > fdb4f5e > > contrib/views/wfmanager/src/main/resources/ui/app/components/flow-designer.js > 37992be > > contrib/views/wfmanager/src/main/resources/ui/app/domain/bundle/bundle-xml-importer.js > b3eed4b > > contrib/views/wfmanager/src/main/resources/ui/app/domain/coordinator/coordinator-xml-importer.js > 12d923c > > contrib/views/wfmanager/src/main/resources/ui/app/domain/workflow-importer.js > a076e3e > > contrib/views/wfmanager/src/main/resources/ui/app/templates/components/designer-errors.hbs > 3cab194 > > > Diff: https://reviews.apache.org/r/57505/diff/1/ > > > Testing > ------- > > Manual > > > Thanks, > > Madhan Reddy > >
