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.

Sign up for GitHub

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

Jump to bottom

fix: import { Route } from 'umi'; #2567

Merged

sorrycc merged 2 commits into umijs:master from imhele:fix-4406

Jun 10, 2019

Contributor

imhele commented Jun 10, 2019

Checklist

npm test passes
tests are included
documentation is changed or added
commit message follows commit guidelines

Description of change

any feature?
close umi 当中没有 Route 的定义,导至AuthorizedRoute.tsx报错 ant-design/ant-design-pro#4406

imhele added 2 commits

June 10, 2019 17:03


          Update index.d.ts

bd8b30f


          Update package.json

798f027

coveralls commented Jun 10, 2019

Pull Request Test Coverage Report for Build 3925

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.481%

Totals
Change from base Build 3919:	0.0%
Covered Lines:	1448
Relevant Lines:	4084

💛 - Coveralls

1 similar comment

coveralls commented Jun 10, 2019

Pull Request Test Coverage Report for Build 3925

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.481%

Totals
Change from base Build 3919:	0.0%
Covered Lines:	1448
Relevant Lines:	4084

💛 - Coveralls

sorrycc changed the title ~~Fix: import { Route } from 'umi';~~ fix: import { Route } from 'umi';

sorrycc added the pr(bug) label

sorrycc merged commit 4b0e0b1 into umijs:master

imhele deleted the fix-4406 branch

June 10, 2019 10:53

weekly-digest bot mentioned this pull request

Weekly Digest (9 June, 2019 - 16 June, 2019) #2614

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels