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

fix: read project_id from cat #2374

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

tangcent
Copy link
Contributor

更新时project_id非必须字段,更新接口作为openapi需对此保持兼容.
故当project_id不存在时,尝试从cat_idproject.


fix #2371

@tangcent
Copy link
Contributor Author

@ariesly15 PTAL

@MrZbb
Copy link

MrZbb commented Jan 10, 2022

是不是原作者不维护了,这个tag的报错最新版本还是存在

@tangcent
Copy link
Contributor Author

是的,又没人维护了

@oubeichen
Copy link

@hellosean1025 希望尽快处理这个问题

@shcw
Copy link

shcw commented Feb 21, 2022

使用最新版本提交带有tag的 保存还是不成功

@tangcent
Copy link
Contributor Author

@Shencw 你是指合并这个PR还是无法保存吗?

@shcw
Copy link

shcw commented Feb 25, 2022

@Shencw 你是指合并这个PR还是无法保存吗?

我的是提示错误,但实际成功了。 使用了最近的master分支也不行。

我这样解决的问题 #2371 (comment)

@oubeichen
Copy link

@Shencw 你是指合并这个PR还是无法保存吗?

我的是提示错误,但实际成功了。 使用了最近的master分支也不行。

我这样解决的问题 #2371 (comment)

这个PR就包含了你说的那个方案,我自行合并后使用一切正常。
就是官方一直没人维护,导致最新版本一直不可用,难办。

@chentaolian
Copy link

1.10.2 存在bug,更新接口如果存在tag标签值,更新成功,提示失败

原因:controllers/interface.js::autoAddTag方法,更新project记录需要project_id,通过up接口前端传参params没有project_id字段,导致projectModel更新报错
image
image

解决方法:up方法调用autoAddTag方法前,params赋值project_id, params.project_id = interfaceData.project_id;

image

@yangqiang
Copy link

1.12.0版本保存接口时,没必要自动保存标签,因为添加新标签的时候已经调用标签接口保存过了。
可以将 server.controllers.interface.js#autoAddTag 中的
if (tags && Array.isArray(tags) && tags.length > 0)
直接改成
if (params.project_id && tags && Array.isArray(tags) && tags.length > 0)

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.

升级到最新版本后修改文档时报错
6 participants