【代码评审修改】触发器 异步 Http 改成 Http 回调

Author: cooling-jason

src/components/SimpleProcessDesignerV2/src/consts.ts

@@ -756,20 +756,19 @@ export enum TriggerTypeEnum {
    * 发送 HTTP 请求触发器
    */
   HTTP_REQUEST = 1,
-  // TODO @jason：要不把 FORM_UPDATE、FORM_DELETE 调整从 10、11 这样？
   /**
-   * 表单数据更新触发器
+   * 发起 HTTP 回调请求触发器
    */
-  FORM_UPDATE = 2,
+  HTTP_CALLBACK = 2,
   /**
-   * 表单数据删除触发器
+   * 表单数据更新触发器
    */
-  FORM_DELETE = 3,
-  // TODO @jason：1）HTTP_CALLBACK 这个枚举值？然后，相关注释，朝着“HTTP 回调”走；2）枚举值改成 2？
+  FORM_UPDATE = 10,
   /**
-   * 发起异步 HTTP 请求
+   * 表单数据删除触发器
    */
-  ASYNC_HTTP_REQUEST = 4
+  FORM_DELETE = 11
+ 
 }
 
 /**
@@ -804,7 +803,7 @@ export type FormTriggerSetting = {
 
 export const TRIGGER_TYPES: DictDataVO[] = [
   { label: 'HTTP 请求', value: TriggerTypeEnum.HTTP_REQUEST },
-  { label: '异步 HTTP 请求', value: TriggerTypeEnum.ASYNC_HTTP_REQUEST },
+  { label: 'HTTP 回调', value: TriggerTypeEnum.HTTP_CALLBACK },
   { label: '修改表单数据', value: TriggerTypeEnum.FORM_UPDATE },
   { label: '删除表单数据', value: TriggerTypeEnum.FORM_DELETE }
 ]

src/components/SimpleProcessDesignerV2/src/nodes-config/TriggerNodeConfig.vue

@@ -36,11 +36,9 @@
           </el-select>
         </el-form-item>
         <!-- HTTP 请求触发器 -->
-        <!-- TODO @jason：可以考虑用 [].include 更简洁 -->
         <div
           v-if="
-            (configForm.type === TriggerTypeEnum.HTTP_REQUEST ||
-              configForm.type === TriggerTypeEnum.ASYNC_HTTP_REQUEST) &&
+            [TriggerTypeEnum.HTTP_REQUEST, TriggerTypeEnum.HTTP_CALLBACK].includes(configForm.type) &&
             configForm.httpRequestSetting
           "
         >
@@ -407,9 +405,9 @@ const changeTriggerType = () => {
     return
   }
 
-  if (configForm.value.type === TriggerTypeEnum.ASYNC_HTTP_REQUEST) {
+  if (configForm.value.type === TriggerTypeEnum.HTTP_CALLBACK) {
     configForm.value.httpRequestSetting =
-      originalSetting?.type === TriggerTypeEnum.ASYNC_HTTP_REQUEST &&
+      originalSetting?.type === TriggerTypeEnum.HTTP_CALLBACK &&
       originalSetting.httpRequestSetting
         ? originalSetting.httpRequestSetting
         : {
@@ -566,7 +564,7 @@ const getShowText = (): string => {
   let showText = ''
   if (
     configForm.value.type === TriggerTypeEnum.HTTP_REQUEST ||
-    configForm.value.type === TriggerTypeEnum.ASYNC_HTTP_REQUEST
+    configForm.value.type === TriggerTypeEnum.HTTP_CALLBACK
   ) {
     showText = `${configForm.value.httpRequestSetting?.url}`
   } else if (configForm.value.type === TriggerTypeEnum.FORM_UPDATE) {