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

feat: use a prettier logo #3

Merged
merged 4 commits into from
Apr 7, 2024
Merged

feat: use a prettier logo #3

merged 4 commits into from
Apr 7, 2024

Conversation

skypesky
Copy link
Collaborator

@skypesky skypesky commented Apr 7, 2024

关联 Issue

主要改动

  • 修复了: title 书写不规范的问题
  • 修复了: README.md 链接展示不对的问题
  • 改进了: 使用更好的 icon

界面截图

测试计划

检查清单

  • 这次变更包含 breaking change,我为 breaking change 编写了 migration script【如果不是 breaking change 可以勾选】
  • 本次变更新增了文件需要被包含在 npm 包的文件,且对应 package.json 的 files 字段包括了这些新增的文件
  • 本次变更需要更新文档,并且我更新了相关文档,如果还没更新文档,请新建文档更新的 Issue 并关联上来
  • 本次变更的地方已经有测试覆盖,并且我调整了变更部分的测试覆盖
  • 本次变更新增的代码逻辑也增加了测试覆盖
  • 本次变更的兼容性测试覆盖了移动端【手机浏览器、钱包内置浏览器】
  • 本次变更的兼容性测试覆盖了 Chrome
  • 本次变更的兼容性测试覆盖了 Safari
  • 本次变更的兼容性测试覆盖了 PC 端
  • 本次变更中有添加或更新 npm 依赖,并且没有导致同 1 个依赖出现多个版本【检查 yarn.lock 的 diff: npx yarn-deduplicate yarn.lock
  • 本次变更我已经把 ArcBlock 的依赖升级到了最新:npm run update:deps
  • 该做 i18n 的地方已经增加了 i18n,包括后端的报错
  • 后端代码的抛错的错误消息比较明确,并且记录了日志
  • 后端中间件里应该有 try-catch 的地方都做了处理
  • 本次变更 request headers 新增了字段,需要 AWS CDN 放行一下

@skypesky skypesky changed the title WIP: fix: link error feat: use a prettier logo Apr 7, 2024
@skypesky skypesky requested a review from wangshijun April 7, 2024 11:41
@skypesky skypesky merged commit 7b24393 into main Apr 7, 2024
3 checks passed
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.

None yet

2 participants