Skip to content
This repository has been archived by the owner on Jul 5, 2022. It is now read-only.

⭐️: 单条件步骤支持 #61

Merged
merged 2 commits into from
Mar 15, 2022

Conversation

JayWenStar
Copy link
Contributor

等下开会,接口数据形式、规则晚点补充

@JayWenStar
Copy link
Contributor Author

JayWenStar commented Mar 15, 2022

  • 增加if else步骤后公共步骤的一些规则

    • 比如id为1的if步骤,底下有2,3两个子步骤,那2,3不能被加入到公共步骤中;但是2,3可以是公共步骤的id
    • 如果把id为1的if步骤添加到公共步骤,那执行的时候如果if通过,会把子步骤2,3也给执行了
  • 目前测试用例的步骤详情接口返回格式如下,id=76、id=80的步骤都是if步骤,且76嵌套80:

    • image
  • 对于公共步骤列表页,基本同上:

    • image
  • 对于公共步骤的步骤列表页,也基本同上,只是好像因为多了一列数据而无法显示:

    • image
  • 关于不同层级之间的拖拉排序暂时没有很好的方案,目前能想到的就是sort的值交给前端控制,每一层if都是从1开始,移动后直接更新后端的sort,不再需要stepSort接口。

Copy link
Member

@ZhouYixun ZhouYixun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

先合并

@ZhouYixun ZhouYixun merged commit 2c8479f into SonicCloudOrg:v1.3.2-beta Mar 15, 2022
@ZhouYixun
Copy link
Member

我是打算单独查子步骤,这样排序可以套用现在的

@JayWenStar
Copy link
Contributor Author

我是打算单独查子步骤,这样排序可以套用现在的

“单独”查子步骤没理解,我感觉套用不了,按你说的那种方式,我猜会变成这样:

  • 步骤1的sort=1
  • 步骤2的sort=2
    • 步骤3是2的子步骤,sort=3
    • 步骤3是2的子步骤,sort=4

现在要把步骤2放到1前面,步骤1的sort=4,步骤2的sort=1,步骤3的sort=2,步骤4的sort=3,多重嵌套后遍历感觉只会更复杂。


如果前端对每一层都单独排序,就不用那么麻烦了,像下面这样:

  • 步骤1的sort=1
  • 步骤2的sort=2
    • 步骤3是2的子步骤,sort=1
    • 步骤3是2的子步骤,sort=2

步骤1和步骤2交换根本不需要管子步骤的sort,就算子步骤和父步骤拖拽交换,比如子步骤12345中拖出了3(后称子3),那子3之后的sort全-1即可;拖出来的子3插入到父级12的父1后面,那么子3的sort就等于父1的sort+1,子3后的sort均+1即可。

@ZhouYixun
Copy link
Member

我是打算单独查子步骤,这样排序可以套用现在的

“单独”查子步骤没理解,我感觉套用不了,按你说的那种方式,我猜会变成这样:

  • 步骤1的sort=1

  • 步骤2的sort=2

    • 步骤3是2的子步骤,sort=3
    • 步骤3是2的子步骤,sort=4

现在要把步骤2放到1前面,步骤1的sort=4,步骤2的sort=1,步骤3的sort=2,步骤4的sort=3,多重嵌套后遍历感觉只会更复杂。

如果前端对每一层都单独排序,就不用那么麻烦了,像下面这样:

  • 步骤1的sort=1

  • 步骤2的sort=2

    • 步骤3是2的子步骤,sort=1
    • 步骤3是2的子步骤,sort=2

步骤1和步骤2交换根本不需要管子步骤的sort,就算子步骤和父步骤拖拽交换,比如子步骤12345中拖出了3(后称子3),那子3之后的sort全-1即可;拖出来的子3插入到父级12的父1后面,那么子3的sort就等于父1的sort+1,子3后的sort均+1即可。

可是其实步骤1和2的sort交换就行了,我不用管三和四的sort,因为他们跟着2的

@JayWenStar JayWenStar deleted the single_condition branch March 18, 2022 12:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants