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

完善 issue 模板 #3360

Merged
merged 1 commit into from
May 15, 2024
Merged

完善 issue 模板 #3360

merged 1 commit into from
May 15, 2024

Conversation

Fangliding
Copy link
Member

本来希望大家都会看说明 结果加了模板之后发现基本都把说明要求提供config当耳边风 然后日志没加必填也基本全是no response
没办法 全部加上了 还弄了了一个代码块和折叠 免得看到辣眼睛的复制 这样可能好一点点

@Fangliding
Copy link
Member Author

@yuhan6665
Copy link
Member

有点严苛 我的意见是鼓励发 issue 资料不足可以再加 比如机场用户无法取得服务端日志

@Fangliding
Copy link
Member Author

Fangliding commented May 14, 2024

其实说辞和之前都是一样的 就是把双端日志配置分开单放了 不然每个issue大概率见面三板斧 问日志问配置搞的跟头文件一样 每个文件都要声明一次 避免这个问题也是加模板的初衷
至于机场用户那是来群里都会轰被走(这里不是机场客服群 去找机场主 巴拉巴拉) 另一方面服务端都摸不到还排查问题除非非常明确不然确实和算命无异

@yuhan6665
Copy link
Member

客户端崩溃这种问题还是有用的 那就把两个日志和配置改成非必要

@Fangliding
Copy link
Member Author

改成非必要估计也是一堆人当耳边风了 这样怎么样()

@yuhan6665
Copy link
Member

没变化。。

@Fangliding
Copy link
Member Author

image
加了一小行 不知道wsm为什么github compare不出来

@yuhan6665 yuhan6665 merged commit b98d060 into main May 15, 2024
@yuhan6665
Copy link
Member

行吧 👍

@yuhan6665 yuhan6665 deleted the issue-temp branch May 20, 2024 23:42
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.

None yet

2 participants