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: testcase pages show badge question #462

Merged
merged 1 commit into from
May 29, 2024

Conversation

SamYSF
Copy link
Contributor

@SamYSF SamYSF commented May 28, 2024

Fixes #461

fix testcase pages show badge when badge value is 0.

@LinuxSuRen
Copy link
Owner

It would be much clear if you could post a screenshot of the changes.

@LinuxSuRen LinuxSuRen added bug Something isn't working ospp 开源之夏 https://summer-ospp.ac.cn/ labels May 28, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@LinuxSuRen LinuxSuRen merged commit b9533ee into LinuxSuRen:master May 29, 2024
14 checks passed
@LinuxSuRen
Copy link
Owner

image

这里好像缺了一个 badge 来提示有 Body 返回。另外,现在没有地方展示接口调用的耗时信息, @SamYSF 有没有好主意啊。

@SamYSF
Copy link
Contributor Author

SamYSF commented May 29, 2024

这里好像缺了一个 badge 来提示有 Body 返回。另外,现在没有地方展示接口调用的耗时信息
1.我看了一下现在前端 body 定义的类型是字符串,那么可以加入字符串不为空则 badge 提示有返回。
2.现在已经有接口调用的耗时信息了吗?如果有的话我认为加在"输出"内即可。
image

@LinuxSuRen
Copy link
Owner

嗯,在输出日志中打印出来也可以

@SamYSF SamYSF deleted the fix-pages-show branch June 14, 2024 10:40
LinuxSuRen pushed a commit that referenced this pull request Jun 17, 2024
* chore(deps): update jumpserver/jms_all docker tag to v3.7.1

* Update app version [skip ci]

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: github-action update-app-version <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ospp 开源之夏 https://summer-ospp.ac.cn/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] front testcase pages show question
2 participants