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

modules/tagsView.js 删除缓存有误 #367

Closed
tlx6550 opened this issue Dec 22, 2017 · 3 comments
Closed

modules/tagsView.js 删除缓存有误 #367

tlx6550 opened this issue Dec 22, 2017 · 3 comments

Comments

@tlx6550
Copy link
Contributor

tlx6550 commented Dec 22, 2017

经测试,在store中 modules/tagsView.js,删除缓存页面原写法有问题
源代码const index = state.cachedViews.indexOf(i)
state.cachedViews = state.cachedViews.slice(index, i + 1)
应为
const index = state.cachedViews.indexOf(i)
state.cachedViews = state.cachedViews.slice(index, index + 1)
博主看看是不是这样呢

@PanJiaChen
Copy link
Owner

你说的是对的,是我的疏忽。之前合别人的代码的时候没有做好codereview。
方便提一个pr么?

@PanJiaChen
Copy link
Owner

#368

@jiangjianshuang
Copy link

我做了几个页面,只有一个页面可以缓存,其它2个页面都不可以不知道是不是页面内容太多了,一个页面分页可以保存,另外一个页面进去就执行created()

这个是不是有内容限制,我把那个可以的页面复制到另外一个页面不行的页面也可以

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

No branches or pull requests

3 participants