codefever icon indicating copy to clipboard operation
codefever copied to clipboard

功能问题

Open Rabbit0w0 opened this issue 3 years ago • 10 comments

Pull Requests

  • PR数量无法被正确统计
  • 审计PR后必须刷新页面
  • 不论结果如何都会显示合并失败

Commits

  • 切换分支时不会显示相应分支的提交数
  • 查看提交时链接内始终是缩短后的commit id 在项目较大时较为危险

Suggestions

建议添加issues等功能

Rabbit0w0 avatar Apr 09 '22 13:04 Rabbit0w0

@Rabbit0w0 您好,针对你的问题,我集中回答一下。

Pull Requests

  1. PR 数量无法正确统计具体是什么意思,你期望得到什么样的结果呢,在合并请求页面中,Codefever 对于 未合并 和 已合并 的项目是分开展示的,你说的是菜单栏的数字没有变化么。
  2. 我们经过测试,评审按钮点击后,评审环节会结束。不需要刷新页面。
  3. 如果 你尝试合并的 commit 已经在目标分支中存在,合并会出错 (实际上已经合并就不能重复合并)

Commits

  1. 左侧菜单的数字不会随着操作的变化而变化,只显示默认分支的相关数据。
  2. commit hash 我们目前以长度 8 字符串显示,实际上所有操作都是以长度 40 字符串来操作的。这块不需要担心

Suggestions

  1. 我们的产品线中已经有 issue tracking 和 project management 类产品 (Tracup),因此,不会在 codefever 中新增 issue tracking 功能,在计划中我们会增强 Tracup 和 CodeFever 两个产品之间的关联程度。

很感谢你的建议 和 反馈,如果上述的描述的行为或前提和你实际遇到的情况不符,请直接回复此 issue,我们会标记为 bug 进行处理。 很可能需要你帮助你提供一些细节信息。

cubicwork avatar Apr 11 '22 03:04 cubicwork

@.@.

合并时出现的第一个错误是合并出错(实际上已经合并) 第二个则是参数错误

Rabbit0w0 avatar Apr 11 '22 13:04 Rabbit0w0

@.@. 合并时出现的第一个错误是合并出错(实际上已经合并) 第二个则是参数错误

@Rabbit0w0 您好,我没有太理解你想描述的内容,你是想说 合并的时候 你点了合并按钮,页面没有任何变化,但是合并已经进行了。 当你再次点合并按钮的时候 就提示了合并出错的错误(实际上已经合并),这个时候 再次点击 合并按钮 就会提示参数错误。是这样么?

那其余的问题是否还有不符合预期的地方呢? 如果我的理解不正确,请在 issue 中告知。

cubicwork avatar Apr 12 '22 01:04 cubicwork

@cubicwork docker实测,点击合并,直接弹窗提示合并失败,然后出现在合并请求-已打开列表中,此时点击进去,再点击合并,仍旧提示合并失败,等待一段时间后,手动刷新页面,会出现在已合并列表中。是否是缓存刷新有问题?

aohun avatar May 20 '22 07:05 aohun

@aohun 这块我们需要再次复现确认下,大概率可能是合并过程比较慢,前端还在等待后端结果返回。

cubicwork avatar May 20 '22 08:05 cubicwork

可以先显示正在合并中,然后返回到合并列表,最后 后台刷新合并状态。

aohun avatar May 20 '22 15:05 aohun

@aohun @cubicwork @Rabbit0w0 关于合并出错的问题可能原因是由于项目较大时 merge 需要耗费更多资源,在 CPU 算力不足时请求可能会超时。解决方法如下:

  • 登录 docker shell (如果使用 docker 方式部署)
  • 打开 /usr/local/nginx/conf/nginx.conf 文件
  • 定位到 78 行
  • 在 78 后新增一行 fastcgi_read_timeout 300s;
  • 保存文件
  • 运行 service nginx reload 重启服务 即可

具体可以参照 #89

cubicwork avatar May 24 '22 09:05 cubicwork

@cubicwork 按照解决方法操作后,提交合并提示 创建合并请求失败,请稍后重试,二次点击提示 存在已打开的合并请求

aohun avatar May 27 '22 08:05 aohun

@cubicwork 请问bug修复了吗?

aohun avatar Jun 14 '22 05:06 aohun

@cubicwork 不好意思,我们还在检查原因,我们并没有复现出您描述的内容。具体还在继续积极查找原因。

cubicwork avatar Jun 14 '22 08:06 cubicwork