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

[WIP] Update website #62

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[WIP] Update website #62

wants to merge 1 commit into from

Conversation

mave99a
Copy link

@mave99a mave99a commented Nov 13, 2022

! DO NOT MERGE, WORKING IN PROGRESS

关联 Issue

主要改动

界面截图

测试计划

检查清单

  • 本次变更新增了文件需要被包含在 npm 包的文件,且对应 package.json 的 files 字段包括了这些新增的文件
  • 本次变更需要更新文档,并且我更新了相关文档,如果还没更新文档,请新建文档更新的 Issue 并关联上来
  • 本次变更的地方已经有测试覆盖,并且我调整了变更部分的测试覆盖
  • 本次变更的兼容性测试覆盖了 PC 端
  • 本次变更的兼容性测试覆盖了移动端【手机浏览器、钱包内置浏览器】
  • 本次变更中有用户输入的逻辑,用户输入的后端、前端都增加了校验、错误提示
  • 本次变更中有添加或更新 npm 依赖,并且没有导致同 1 个依赖出现多个版本【检查 yarn.lock 的 diff】
  • 本次变更我已经把 ArcBlock 的依赖升级到了最新:https://www.notion.so/arcblock/JS-afb6960ba36546fa8ddc638fc47e5f51

@mave99a mave99a changed the title [DO NOT Merge] Update website [WIP] Update website Nov 13, 2022
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.

1 participant