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.

Sign up for GitHub

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

Jump to bottom

feat(service): ✨ http错误提示 #184

Merged
merged 4 commits into from
Jan 15, 2025
Merged

feat(service): ✨ http错误提示 #184

merged 4 commits into from
Jan 15, 2025

Conversation

Aybrea
Copy link
Contributor

@Aybrea Aybrea commented Jan 14, 2025

💻 变更类型 | Change Type

  • ✨ feat | 新增功能
  • 🐛 fix | 修复缺陷
  • ♻️ refactor | 代码重构（不包括 bug 修复、功能新增）
  • 💄 style | 代码格式（不影响功能，例如空格、分号等格式修正）
  • 📦️ build | 构建流程、外部依赖变更（如升级 npm 包、修改 vite 配置等）
  • 🚀 perf | 性能优化
  • 📝 docs | 文档变更
  • 🧪 test | 添加疏漏测试或已有测试改动
  • ⚙️ ci | 修改 CI 配置、脚本
  • ↩️ revert | 回滚 commit
  • 🛠️ chore | 对构建过程或辅助工具和库的更改（不影响源文件、测试用例）

🔀 变更说明 | Description of Change

UI提示请求失败的api url

📝 补充信息 | Additional Information

@github-actions github-actions bot added the 前端 关于前端的代码修改 label Jan 14, 2025
@codecov Codecov
Copy link

codecov bot commented Jan 14, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 0% with 63 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/services/http.ts 0.00% 37 Missing ⚠️
src/common/exception.ts 0.00% 26 Missing ⚠️
Files with missing lines Coverage Δ
src/common/exception.ts 0.00% <0.00%> (-50.00%) ⬇️
src/services/http.ts 0.00% <0.00%> (-4.09%) ⬇️

... and 33 files with indirect coverage changes

@nongyehong nongyehong self-requested a review January 14, 2025 07:07
nongyehong
nongyehong reviewed Jan 14, 2025
View reviewed changes
Copy link
Member

@nongyehong nongyehong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

封装在AppException类中继续完善一下

@nongyehong nongyehong linked an issue Jan 15, 2025 that may be closed by this pull request
[feat] 封装全局http错误提示，拦截错误请求并且重试 #187
Closed
@nongyehong nongyehong merged commit cd03444 into HuLaSpark:master Jan 15, 2025
10 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nongyehong nongyehong nongyehong approved these changes

Assignees
No one assigned
Labels
前端 关于前端的代码修改
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feat] 封装全局http错误提示，拦截错误请求并且重试
2 participants
@Aybrea @nongyehong