Skip to content

Conversation

matijagaspar
Copy link
Contributor

Require velocity only on client, make a stub on server.

This fixed most if not all issues with ant-design components not working on a project that does server side rendering.

@matijagaspar
Copy link
Contributor Author

The remaining lint errors are in the examples which I did not modify. The issue is "Declare only one React component per file react/no-multi-comp" and to solve it, the examples would need to be properly refactored, so I think this is beyond the scope of this fix. As for the reduced coverage, since the fix is dealing with server side rendering, there is no way I can write a browser test to test server side render.

afc163 added a commit that referenced this pull request Jan 13, 2016
@afc163
Copy link
Member

afc163 commented Jan 13, 2016

Thx for your contribution!!! I fixed the lint problems and the coverage issue does not matter for this.

afc163 added a commit that referenced this pull request Jan 13, 2016
@afc163 afc163 merged commit 12da7b1 into react-component:master Jan 13, 2016
@afc163
Copy link
Member

afc163 commented Jan 13, 2016

0.11.4 bumped.

@matijagaspar matijagaspar deleted the fix-server-side-rendering branch January 13, 2016 16:24
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.

2 participants