Repository navigation
feat(analyze): 按分类查看基准的选择器与范围标识 - #489
Open
GenevieneTrailblazer wants to merge 3 commits into
Open
GenevieneTrailblazer wants to merge 3 commits into
GenevieneTrailblazer wants to merge 3 commits into
Conversation
新增分类选择器，把选中的分类写进地址栏并随查询一起提交；新增范围标识组件，在页面上写清当前看到的统计是相对哪个范围算出来的。两个组件都配有测试。 Signed-off-by: GenevieneTrailblazer <GenevieneTrailblazer@users.noreply.github.com>
汇总查询的变量里加入分类，取数时一并提交；接线测试确保调用方真的把分类传了下去，避免参数写好却没有任何调用方使用。 Signed-off-by: GenevieneTrailblazer <GenevieneTrailblazer@users.noreply.github.com>
四个汇总查询的文档与对应类型加入可选的分类变量，并同步到生成物。 Signed-off-by: GenevieneTrailblazer <GenevieneTrailblazer@users.noreply.github.com>
|
@GenevieneTrailblazer is attempting to deploy a commit to the codersett's projects Team on Vercel. A member of the Team first needs to authorize it. |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
解决问题
这是 #325 在前端这一层的落实。用户可以选定一个项目分类，页面上的统计随即按这个分类的成员
重新计算，并且清楚地标出当前看到的范围。它与汇总侧 #309 以及读取侧 #456 一起构成完整的一条链。
具体做法
新增了一个分类选择器，选中的分类会写进地址栏，并随查询一起提交。同时新增了一个范围标识组件，
在页面上把当前比较的范围写清楚，避免把某个已收录的分类误当成全部项目。
汇总取数的环节把分类变量一并传下去。这一步还补了一条接线测试，专门用来确认调用方真的使用了
这个参数。要防的是一种已经出现过的问题：参数与逻辑都写好了，却没有任何调用方传它，页面看起来
一切正常，实际永远在使用全局范围。
四个汇总查询的文档与对应的类型也加入了可选的分类变量，两者同步更新。
验证
前端测试会在持续集成里运行，如果结果不理想我会继续跟进。这个改动涉及的四个查询在文档与类型
之间保持一致，这一点已经逐项核对过。
这个 PR 不包含的部分
诊断页面以及产品内的解释性分析旁路，会在后续单独提交，因为它们依赖上游还没有的接口字段。
关于 AI 的使用
这份改动由 AI 助手协助起草，提交之前已经过人工逐条审核。如果需要查看审核记录，我们可以提供。