perf: cancel the default validate of user-info to show page

Drowned-fish commented 3 years ago

Types of changes

[ ] New feature
[ ] Bug fix
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[x] Performance improvement
[ ] Test cases
[ ] Continuous integration
[ ] Typescript definition change
[ ] Breaking change

Background and context

user-info页面默认校验了user信息返回null，导致无法看到默认的效果。

Solution

How is the change tested?

Changelog

| Changelog(CN) | Changelog(EN) | Related issues | |取消user-info页面对user的校验 | Cancel the default verification of user information | -------------- |

Checklist:

[ ] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others should be submitted to master branch)

Other information

Drowned-fish commented 3 years ago

有人看PR吗

PengJiyuan commented 3 years ago

有人看PR吗

抱歉回复晚了，感谢提交～明天 @kirazxyun 看一下，目前修改需要同时修改 4 个模版，我们需要把这个过程简化一下

kirazxyun commented 3 years ago

@Drowned-fish 你好，非常感谢提交 pr，我的想法是在没有用户信息的时候不应该显示页面内详情，因为我们的页面内详情通常情况下需要根据用户信息去请求，所以才会在这里阻断页面的显示。很乐意和您交流一下想法。

Drowned-fish commented 3 years ago

@Drowned-fish 你好，非常感谢提交 pr，我的想法是在没有用户信息的时候不应该显示页面内详情，因为我们的页面内详情通常情况下需要根据用户信息去请求，所以才会在这里阻断页面的显示。很乐意和您交流一下想法。

@kirazxyun 我是这么想的，类似pro这种项目，开发者在成功运行之后会先看看这个项目有哪些内容，每个页面模块具体长什么样。所以内容才是吸引开发者去使用。上次我成功运行项目之后，发现我只能看登录页，想看其他页面还得去看代码，然后改代码。然后在用户信息这个页面也是一样，我还以为是可能有mock接口我没开启，去看接口请求，最后才发现页面直接返回null了。

kirazxyun commented 3 years ago

@Drowned-fish 你好，非常感谢提交 pr，我的想法是在没有用户信息的时候不应该显示页面内详情，因为我们的页面内详情通常情况下需要根据用户信息去请求，所以才会在这里阻断页面的显示。很乐意和您交流一下想法。

@kirazxyun 我是这么想的，类似pro这种项目，开发者在成功运行之后会先看看这个项目有哪些内容，每个页面模块具体长什么样。所以内容才是吸引开发者去使用。上次我成功运行项目之后，发现我只能看登录页，想看其他页面还得去看代码，然后改代码。然后在用户信息这个页面也是一样，我还以为是可能有mock接口我没开启，去看接口请求，最后才发现页面直接返回null了。

@Drowned-fish 明白，收到您的想法十分感谢。以下是回复，占用您一点时间交流一下～

登陆这个放在一开始确实不是个很好的体验，对于用户继续往下使用确实有理解成本，这个问题我这边会改一下，将首页定位到工作台，让程序一开始就能看到所有的页面。
用户信息页面为什么出不来，这个我很好奇原因～，用户信息在程序一开始就去请求的，数据存放在全局状态里，理论上来讲如果 mock 生效不应该出现用户信息不存在的问题（这个现象是不是我们能一起看一下问题出在哪里呢？）
因为 pro 模拟的是一个真实的项目，至少我们逻辑上要去做靠近，所以不去管用户信息是否存在就显示页面，有点不是那么靠谱，但是直接返回 null 确实暴力了点，我觉得做个提示会不会更好（比如没有获取到用户信息，请刷新重试的显示信息）

arco-design / arco-design-pro