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

Dva2.0 #567

Merged
merged 4 commits into from
Sep 9, 2017
Merged

Dva2.0 #567

merged 4 commits into from
Sep 9, 2017

Conversation

yangbin1994
Copy link
Contributor

有一些格式化引起diff,抱歉。。。
升级dva@2;使用cross-env;给Loader组件增加了一个fullScreen选项,为了在按需加载pageRoute的时候有一个loading效果

@yangbin1994 yangbin1994 mentioned this pull request Sep 8, 2017
@zuiidea
Copy link
Owner

zuiidea commented Sep 8, 2017

可以可以,我先测一下有没有额外的bug,再merge

@zuiidea
Copy link
Owner

zuiidea commented Sep 8, 2017

第一次载入时样式还没加载上会闪烁

@yangbin1994
Copy link
Contributor Author

开发环境css是js生成的,到生产环境就好了

@zuiidea zuiidea merged commit 1288853 into zuiidea:master Sep 9, 2017
@yangbin1994 yangbin1994 deleted the dva2.0 branch September 9, 2017 17:15
@Gavinchen92
Copy link

Gavinchen92 commented Sep 15, 2017

<Route exact path="/" render={() => (<Redirect to="/dashboard" />)} />
我现在的项目也模仿着你的pr升级到dva2.0了, 但是这条重定向路由一直不生效, 已经卡了半天时间了
@yangbin1994

@@ -1,7 +1,7 @@
import React from 'react'
import PropTypes from 'prop-types'
import { Menu, Icon } from 'antd'
import { Link } from 'dva/router'
import { LinkWrapper } from 'components'
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的 LinkWrapper 是从哪里来的呀,我查看了下 src/components/index.js 中导出的对象中并没有 LinkWrapper,有点疑问,请教一下。

@amenzai amenzai mentioned this pull request Sep 13, 2018
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.

4 participants