dropByCacheKey 需要同步操作

CJY0208 / react-router-cache-route

Route with cache for react-router V5 like <keep-alive /> in Vue

MIT License

1.16k stars 113 forks source link

Closed ylwbl closed 3 years ago

ylwbl commented 3 years ago

目前由于dropByCacheKey内部使用this.setState,导致了异步的问题,目前已经fork了代码并且此方法传入callBack进行同步操作,是否需要提交pr

ylwbl commented 3 years ago

另外其他的命令式操作是否也需要加入回调函数

CJY0208 commented 3 years ago

原则上不再建议使用 callback，因为存在回调地狱的问题（虽然 drop 场景不太可能地狱）

如果 PR 的话可以是 Promise 形式 —— JS 生态中标准范式的异步解决方案