-
Notifications
You must be signed in to change notification settings - Fork 84
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: 添加响应类型包装器 #190
feat: 添加响应类型包装器 #190
Conversation
@WangMingHua111 有没有响应的示例,是解决了什么问题,目前看出是做了什么修改 |
这种的话,我这边建议时候在 axios(或者其他的 request 请求客户端)的响应流过滤器里面直接处理就可以了。 即,axos 的话 是在 例如你这里的话,那就是 你看下能不能足你需求 |
贴一个我对接的一个工程的例子 axiosInstance.interceptors.response.use(
(response) => {
const config: any = response.config
const data: ResponseBodyType = response.data
if (config) {
hideLoading()
}
if (data.success && data.code === 200) {
return response
} else if (response.data.code === 0) {
return response.data
} else {
const code = data.code
let errorMessage = '' |
能满足要求,似乎是应该这样处理 |
如果你这边不建议包装返回类型 我这边把这个提交关闭掉 |
即在返回值里面自行 |
这仅能算一个一个业务需求,strictNullChecks 会导致整个项目中的代码不进行检查。在这里我希望是进行 api data 进行 bind 的时候不检查。 |
通过 responseTypeWrapper 参数我可以给项目打补丁,不然目前没有任何办法转换 返回的类型。 |
最近有点忘,忘记看了~我再重新评估一下 |
@WangMingHua111 这个pr还有合并的需求吗,如果有我这边重新打开一下 |
重新打开了 |
支持响应类型包装,解决 后端文档与接口返回不一致问题