Skip to content
This repository has been archived by the owner on Nov 17, 2022. It is now read-only.

feat: store initialStates #435

Merged
merged 13 commits into from
Oct 17, 2022
Merged

feat: store initialStates #435

merged 13 commits into from
Oct 17, 2022

Conversation

luhc228
Copy link
Member

@luhc228 luhc228 commented Aug 11, 2022

@luhc228 luhc228 requested review from wssgcg1213 and ClarkXia August 11, 2022 03:04
@luhc228 luhc228 self-assigned this Aug 11, 2022
@vercel
Copy link

vercel bot commented Aug 11, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
ice-v3 ✅ Ready (Inspect) Visit Preview Oct 17, 2022 at 3:07AM (UTC)

@codecov-commenter
Copy link

Codecov Report

Merging #435 (e147f20) into release-next (45dc7a8) will increase coverage by 0.02%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           release-next     #435      +/-   ##
================================================
+ Coverage         82.20%   82.23%   +0.02%     
================================================
  Files                87       87              
  Lines              7448     7448              
  Branches            941      941              
================================================
+ Hits               6123     6125       +2     
+ Misses             1319     1317       -2     
  Partials              6        6              
Impacted Files Coverage Δ
packages/ice/src/service/config.ts 87.03% <0.00%> (+1.23%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@@ -16,6 +16,7 @@
"@types/react": "^18.0.0",
"@types/react-dom": "^18.0.0",
"browserslist": "^4.19.3",
"regenerator-runtime": "^0.13.9"
"regenerator-runtime": "^0.13.9",
"tslib": "^2.4.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

为啥需要这个

Copy link
Member Author

Choose a reason for hiding this comment

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

image

Copy link
Collaborator

@ClarkXia ClarkXia Aug 16, 2022

Choose a reason for hiding this comment

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

所以项目依赖中你会推荐把 tslib 也默认加进去吗

Copy link
Member Author

Choose a reason for hiding this comment

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

image

看 next 也会加 tslib

Copy link
Collaborator

Choose a reason for hiding this comment

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

好多地方都会用到, ts 优先的话我也建议脚手架默认加 tslib 在 devDep

Copy link
Member Author

Choose a reason for hiding this comment

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

或者也可以默认先不依赖 tslib,把可能遇到的错误以及解决方式(安装一下 tslib)放在 faq 中

Copy link
Collaborator

Choose a reason for hiding this comment

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

这里不用增加复杂度了, 直接脚手架里依赖上吧 (开箱即用)

@luhc228 luhc228 requested a review from ClarkXia August 12, 2022 03:25
@ClarkXia
Copy link
Collaborator

可以参考 #519 支持下消费 app data

wssgcg1213
wssgcg1213 previously approved these changes Sep 16, 2022
ClarkXia
ClarkXia previously approved these changes Sep 22, 2022
wssgcg1213
wssgcg1213 previously approved these changes Oct 14, 2022
@luhc228 luhc228 merged commit d985198 into release-next Oct 17, 2022
@luhc228 luhc228 deleted the feat/store-initialStates branch October 17, 2022 07:26
@ClarkXia ClarkXia mentioned this pull request Oct 27, 2022
16 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants