Skip to content
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

fix: 接口/refreshToken改为/refresh-token #765

Merged
merged 1 commit into from
Oct 22, 2023

Conversation

JarvisChao
Copy link
Contributor

@JarvisChao JarvisChao commented Oct 21, 2023

#761 (comment) 頁面切換失效,經測試後是 /refreshToken 寫錯造成的
https://github.com/pure-admin/vue-pure-admin/blob/main/src/api/user.ts#L38
https://github.com/pure-admin/vue-pure-admin/blob/main/src/utils/http/index.ts#L76

另外補充一下,目前沒有任何針對 refresh 錯誤做出對應措施

@xiaoxian521 xiaoxian521 changed the title fix: 修正錯誤白名單接口 fix: 接口refreshToken改为/refresh-token Oct 22, 2023
@xiaoxian521 xiaoxian521 changed the title fix: 接口refreshToken改为/refresh-token fix: 接口/refreshToken改为/refresh-token Oct 22, 2023
@xiaoxian521 xiaoxian521 merged commit a713d37 into pure-admin:main Oct 22, 2023
1 check passed
@xiaoxian521
Copy link
Member

感谢反馈 错误统一处理的话 放在v5.0.0版本发布后吧 后面我统一处理 当然如果您有意向提pr 我们非常欢迎 处理时应尽量简单😊

@JarvisChao JarvisChao deleted the fix/whiteList branch October 23, 2023 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants