Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

【蓝盾-评审会已评审】【PAC】perf:UI方式新增/编辑流水线重构 #8167

Closed
fayewong-v5 opened this issue Dec 20, 2022 · 4 comments
Assignees
Labels
area/ci/backend CI 后端issue area/ci/frontend CI 前端特性issue for approve 产品特性申请审核 kind/feat/product 产品特性 todo todo
Milestone

Comments

@fayewong-v5
Copy link
Contributor

fayewong-v5 commented Dec 20, 2022

【背景】

  • 新增/编辑流水线需同时支持 Code 和 UI 方式编辑
  • UI 方式编辑的流水线也可以开启 PAC 模式
  • 现有版本变量管理和触发器管理交互不太直观,需进行优化

创建/编辑流水线流程中涉及的变更见 #8125 ,这里不再重复,此需求仅说明 UI 方式编辑时的改动

【需求】
1、进入编排页面时,默认 Code 方式 or UI 方式编辑,由用户偏好决定

  • 比如用户在新建的第一个步骤预览模版切换到了 UI 方式查看,则下一步进入编排时也是 UI 方式编排

2、将原有编排中的 stage-1 下的触发器和变量管理入口拆分出去,拆分后的新入口如下图所示:
image

3、触发器管理 tab 如下:
image

  • 新增流水线时,默认添加手动触发器
  • 点击「新增触发器」侧滑打开选择触发器添加界面,交互和现有版本保持一致
  • 点击 触发器名称,侧滑打开触发器编辑界面,交互和现有版本保持一致
  • 可以禁用触发器
    • 已经开启 PAC 模式的流水线,触发器禁用后:
      • 若切换到 Code 方式查看,看到的代码示例:
        on:
          push: disabled
        
      • 若是编辑流水线,禁用前的触发器配置在DB中仍然保留,在蓝盾页面上重新启用时将恢复配置
  • 可以删除触发器

4、流水线变量 tab 在编排右侧展示:
image

5、若当前tab 内容有变更,尚未保存时切换到其他配置 tab,则在 tab 标题增加修改过的提示
image

6、通知页面如下:
image

  • 构建成功/失败时,都可以添加多个通知
  • 新增/修改通知界面如下:
    • 当不需要发送群消息时
      image
    • 当需要发送群消息时,增加 群ID 和 是否 Markdown 的 选项:
      image

7、设置页面如下:
image

8、调试好流水线,点击「发布」后

@fayewong-v5 fayewong-v5 added the kind/enhancement 功能改进特性 label Dec 20, 2022
@fayewong-v5 fayewong-v5 changed the title 【蓝盾-评审会已评审】【PAC】feat:UI方式新增/编辑流水线重构 【蓝盾-评审会已评审】【PAC】perf:UI方式新增/编辑流水线重构 Dec 21, 2022
@fayewong-v5 fayewong-v5 added kind/feat/product 产品特性 for approve 产品特性申请审核 and removed kind/enhancement 功能改进特性 labels Dec 26, 2022
@fayewong-v5 fayewong-v5 added this to the V1.10 milestone Dec 26, 2022
@EchoQT
Copy link
Collaborator

EchoQT commented Dec 28, 2022

image

普通通知人和群通知ID为何中间隔着通知内容?这排版有点奇怪。

@EchoQT
Copy link
Collaborator

EchoQT commented Dec 28, 2022

image

image

-----这两个地方的构建号设置是不是合并一下放到流水线设置中比较好?
-----另外,推荐版本号作为内置变量,那个开关是什么意思?

@EchoQT
Copy link
Collaborator

EchoQT commented Dec 28, 2022

预览yaml好像可以去掉?
用户本来就可以随时切换yaml方式查看和编辑,而且当用户更习惯于UI方式的时候,发布时提供预览yaml的目的是什么呢?

@EchoQT EchoQT assigned fayewong-v5 and unassigned EchoQT Dec 28, 2022
@fayewong-v5 fayewong-v5 modified the milestones: V1.10, V2.0 Feb 2, 2023
@fayewong-v5 fayewong-v5 assigned terlinhe and royalhuang and unassigned fayewong-v5 Jul 7, 2023
@fayewong-v5 fayewong-v5 added area/ci/frontend CI 前端特性issue area/ci/backend CI 后端issue todo todo labels Jul 7, 2023
@fayewong-v5
Copy link
Contributor Author

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci/backend CI 后端issue area/ci/frontend CI 前端特性issue for approve 产品特性申请审核 kind/feat/product 产品特性 todo todo
Projects
None yet
Development

No branches or pull requests

4 participants