Browse Source

[improvement] improve relation between notification strategy and alarm group on ui (#12200)

* [improvement] improve relation between notification strategy and alarm group

* [improvement] change to use ```&&```

Co-authored-by: Jackie <‘2875334588@qq.com’>
fuchanghai 2 years ago
parent
commit
c7d9719e97

+ 2 - 2
dolphinscheduler-ui/src/views/projects/workflow/definition/components/start-modal.tsx

@@ -323,7 +323,7 @@ export default defineComponent({
               clearable
             />
           </NFormItem>
-          <NFormItem
+          {this.startForm.warningType !== 'NONE' && (<NFormItem
             label={t('project.workflow.alarm_group')}
             path='warningGroupId'
           >
@@ -333,7 +333,7 @@ export default defineComponent({
               v-model:value={this.startForm.warningGroupId}
               clearable
             />
-          </NFormItem>
+          </NFormItem>)}
           <NFormItem
             label={t('project.workflow.complement_data')}
             path='complement_data'

+ 2 - 2
dolphinscheduler-ui/src/views/projects/workflow/definition/components/timing-modal.tsx

@@ -378,7 +378,7 @@ export default defineComponent({
               clearable
             />
           </NFormItem>
-          <NFormItem
+          {this.timingForm.warningType !== 'NONE' && ( <NFormItem
             label={t('project.workflow.alarm_group')}
             path='warningGroupId'
           >
@@ -388,7 +388,7 @@ export default defineComponent({
               v-model:value={this.timingForm.warningGroupId}
               clearable
             />
-          </NFormItem>
+          </NFormItem> )}
         </NForm>
       </Modal>
     )