search

alibaba / ice

🚀 ice.js: The Progressive App Framework Based On React（基于 React 的渐进式应用框架）
https://ice.work
MIT License
17.84k stars 2.09k forks source link

getInitialData API 变更 #5063

Open ClarkXia opened 2 years ago

ClarkXia commented 2 years ago

image

获取初始数据在 appConfig 中设置的名称和 ice 中导出的方法命名上有重复，容易造成开发者的困惑和额外的认知成本，建议修复消费初始数据的方式

变更方案

方案 A

直接通过 import { initialData } from 'ice' 的方式导入，直接消费 initialData

方案 B

直接通过 import { useInitialData } from 'ice' 的方式导入，仅限制组件内部使用

imsobear commented 2 years ago

直接 import initialData 会有时机的问题吧？

ClarkXia commented 2 years ago

暂定方案 B

inier commented 2 years ago

感觉还不如原来的，原来就是一个工具函数，B方案要记录2个方法

imsobear commented 2 years ago

感觉还不如原来的，原来就是一个工具函数，B方案要记录2个方法

但是两个 getInitialData 不会感觉很奇怪吗？

imsobear commented 2 years ago

应该只有一个 useInitialData，不需要 withXXX 的

inier commented 2 years ago

本身这个方法做的事情，并不一定是hook把