鹰信车载后台 eagcar-admin

2022-08-15

• 记录者：荣胜
• 代码评审者：朝润、国宇、荣胜、遗森
• 代码编写者：朝润、国宇、荣胜

团队代码规范约定

1. ts定义的接口命名方式使用首字母大写的驼峰，比如ItemType。
2. ts定义的接口里的注释使用/** */代替//。
3. 没内容的标签用单标签结束，也可以通过esLint限制。

待优化

CommonJs

1. 优化CommonJs里的debounce方法，可首次马上调用函数，之后再延迟调用（解决场景：防抖请求列表数据）。(国宇)

CodeReview

src/hooks/menus.tsx

1. 目前首页是login登录页面。（朝润）
2. 拆分成router配置与menus配置。（朝润）
3. icon传参方式修改，传入的icon组件改成icon名字。（荣胜）

src/components/permission-route

1. 权限组件需补充重定向处理。（朝润）

2022-08-16

• 记录者：遗森
• 代码评审者：朝润、国宇、荣胜、遗森
• 代码编写者：朝润、国宇、荣胜

团队代码规范约定

1. 响应 接口定义 根据pages定义的层级来
2. 常量定义用到name属性的按原来方式，不用name属性的可以用枚举
3. 接口返回值必须定义
4. api里面mock等axios config 不用的话需要注释
5. 每个方法之间要换行

待优化

• 图片加上压缩(国宇)

建议意见

api/role-manege

1. get请求，传参放在第二个参数里，不要放params里（国宇）

icon组件

1. 文件名可以不用加icon。（）

src/constants

• 所有都要定义类型（）

src/constants/common

1. 全局状态码常量要定义类型（）
2. 员工那两个常量要放在对应模块（）

src/constants/utils

1. 映射方法的类型定义可以优化（）