-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
feat: add subpackage @sa/alova
#640
Conversation
onBackendFail并非只有token刷新一个作用,该钩子方法是为了给用户提供统一的源于后端失败的处理用的 |
@Azir-11 因为我看 soybean-admin/packages/axios/src/index.ts Lines 51 to 81 in cfaab85
所以我才在 |
onBackendFail是后端请求在业务上表示失败时调用的异步函数 onBackendFail是更突出业务失败用的 |
当然只留onError其实也可以做到onBackendFail里的事,但是我觉得为了保证过渡平滑,还是保留比较好 |
@Azir-11 在修改时我是发现了这段代码,以下两段代码中都对 soybean-admin/src/service/request/index.ts Lines 48 to 93 in cfaab85
soybean-admin/src/service/request/index.ts Lines 97 to 121 in cfaab85
这里我觉得是多余的,而且虽说 // get backend error message and code
if (error.code === BACKEND_ERROR_CODE) {
message = error.response?.data?.msg || message;
backendErrorCode = String(error.response?.data?.code || '');
} 所以我才统一使用了 如果需要分别处理,也可以通过 {
// ...
onError(error, response) {
if (response) {
// 处理后端业务错误
} else {
// 处理网络错误
}
}
} 最后,以上是我只保留 |
确实,你说的也没什么问题,但是还是需要考虑一下现在正在使用的用户的,去掉onBackendFail或许可行,但是起码需要在至少v1.4才能做 |
@Azir-11 我是增量添加一个@sa/alova子包,并不是在@sa/axios上修改的😂,这个结果是和你们创始人沟通决定的 |
同属于请求封装,在大部分钩子相同的情况下,某一些不同,会带来心智负担 |
此pr包含只添加了
@sa/alova
子包,通过77行代码实现了比@sa/axios
更丰富的特性,具体如下:@sa/axios
的封装设计保持了一致onBackendFail
钩子