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

功能问题 #82

Open
Rabbit0w0 opened this issue Apr 9, 2022 · 10 comments
Open

功能问题 #82

Rabbit0w0 opened this issue Apr 9, 2022 · 10 comments
Labels
bug Something isn't working question Further information is requested

Comments

@Rabbit0w0
Copy link

Rabbit0w0 commented Apr 9, 2022

Pull Requests

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

Commits

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

Suggestions

建议添加issues等功能

@cubicwork
Copy link
Contributor

cubicwork commented Apr 11, 2022

@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 cubicwork added the question Further information is requested label Apr 11, 2022
@Rabbit0w0
Copy link
Author

Rabbit0w0 commented Apr 11, 2022 via email

@cubicwork
Copy link
Contributor

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

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

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

@aohun
Copy link

aohun commented May 20, 2022

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

@cubicwork
Copy link
Contributor

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

@aohun
Copy link

aohun commented May 20, 2022

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

@cubicwork
Copy link
Contributor

cubicwork commented May 24, 2022

@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 cubicwork added the bug Something isn't working label May 24, 2022
@aohun
Copy link

aohun commented May 27, 2022

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

@aohun
Copy link

aohun commented Jun 14, 2022

@cubicwork 请问bug修复了吗?

@cubicwork
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants