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

Feat 89 scm table added #108

Merged
merged 23 commits into from Apr 19, 2022
Merged

Conversation

colynn
Copy link
Member

@colynn colynn commented Mar 27, 2022

Which changes (Bug/Feature):

Fixes #89

Special notes for reviewers:

image

image

image

@codecov-commenter
Copy link

codecov-commenter commented Mar 27, 2022

Codecov Report

Merging #108 (3adf3e3) into v1.5.0-feat-app-refactor (574f23c) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@                     Coverage Diff                     @@
##           v1.5.0-feat-app-refactor    #108      +/-   ##
===========================================================
- Coverage                      4.67%   4.56%   -0.11%     
===========================================================
  Files                            10      10              
  Lines                          1454    1489      +35     
===========================================================
  Hits                             68      68              
- Misses                         1384    1419      +35     
  Partials                          2       2              
Impacted Files Coverage Δ
internal/core/pipelinemgr/pipelinemgr.go 0.00% <0.00%> (ø)
internal/core/pipelinemgr/types.go 0.00% <ø> (ø)
internal/core/pipelinemgr/uitls.go 0.00% <0.00%> (ø)
utils/utils.go 11.42% <0.00%> (-0.34%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 574f23c...3adf3e3. Read the comment docs.

@sampsonye
Copy link
Collaborator

“我的应用”-->“添加应用”后列表未刷新,需要手动F5

@sampsonye
Copy link
Collaborator

“系统管理”-->“源代码集成”-->“新建”-->“测试连接” 功能不可用

@sampsonye
Copy link
Collaborator

项目中关联代码仓库时无法选择和修改分支

@colynn
Copy link
Member Author

colynn commented Mar 29, 2022

项目中关联代码仓库时无法选择和修改分支

获取的分支信息来自 我的应用, 需要在我的应用 内先同步分支信息后 才可以选择

@colynn
Copy link
Member Author

colynn commented Mar 29, 2022

“系统管理”-->“源代码集成”-->“新建”-->“测试连接” 功能不可用

可以转为issue

@sampsonye
Copy link
Collaborator

项目中关联代码仓库时无法选择和修改分支

获取的分支信息来自 我的应用, 需要在我的应用 内先同步分支信息后 才可以选择

场景疑惑:分支信息初始化应该在“应用”中,还是应该在“项目”中?

@colynn
Copy link
Member Author

colynn commented Apr 2, 2022

项目中关联代码仓库时无法选择和修改分支

获取的分支信息来自 我的应用, 需要在我的应用 内先同步分支信息后 才可以选择

场景疑惑:分支信息初始化应该在“应用”中,还是应该在“项目”中?

“应用”里维护的是分支的原信息, 具体的分支选择在 “项目” -- “持续部署” 创建流水线 时选择,这样功能没有影响吧

@sampsonye
Copy link
Collaborator

项目中关联代码仓库时无法选择和修改分支

获取的分支信息来自 我的应用, 需要在我的应用 内先同步分支信息后 才可以选择

场景疑惑:分支信息初始化应该在“应用”中,还是应该在“项目”中?

“应用”里维护的是分支的原信息, 具体的分支选择在 “项目” -- “持续部署” 创建流水线 时选择,这样功能没有影响吧

在考虑权限设计时,可能会有如下场景:

  1. 应用的添加仅由管理员进行
  2. 应用的调整仅由应用创建者自己或高权限用户操作
  3. 项目添加应用时,在一个页面(项目绑定应用页面)操作 还是需要同时操作两个页面(应用详情页刷新分支信息 & 项目绑定页面选择分支)

前两者决定我们的应用列表页如何展示和如何操作
第三个主要考虑构建流程的最佳方案

@colynn
Copy link
Member Author

colynn commented Apr 4, 2022

@sampsonye 分支信息 归属上应该属于应用的原信息,也就是在 “我的应用”添加进来后,操作下同步分支,

或是 应用的分支 每次均从 scm的仓库获取?

@sampsonye
Copy link
Collaborator

@sampsonye 分支信息 归属上应该属于应用的原信息,也就是在 “我的应用”添加进来后,操作下同步分支,

或是 应用的分支 每次均从 scm的仓库获取?

我的理解是,“分支信息”不属于应用的关键信息,因此确实可以考虑应用的分支每次动态从SCM仓库获取

@sampsonye
Copy link
Collaborator

/LGTM

@colynn colynn merged commit af72f79 into v1.5.0-feat-app-refactor Apr 19, 2022
@colynn colynn deleted the feat-89-scm-table-added branch April 29, 2022 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

基于“我的项目”- “应用代码” 独立出 “我的应用”的表
3 participants