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.

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

Setup basic SSR #13

Merged
merged 33 commits into from
Mar 30, 2018
Merged

Setup basic SSR #13

merged 33 commits into from
Mar 30, 2018

Conversation

tonyhhyip
Copy link
Contributor

No description provided.

Tony Yip added 27 commits March 12, 2018 14:38
Add file ignore for JetBrains IDE
2b9931c
Use state for better performance
d435d0f
Use map for better control of flow
7769bc5
Use static props for better reading
a7a61c2
Connect redux at high level components
5f57f09
Please eslint
ab389b6
Add key for list
8f10e5b
Transform List info pure component
5845156
Remove console.log
cca235c
Fix for color
03d8215
Add key for listing
328361c
Extract title bar from meeting
da6f82d
Use function to create store
f24cb8f
Remove meaningless connect of redux
96d4cc9
Make router become high order component
8d6baba
Fix warning for List
254d53c
Use style instead of magic spacing
e8f9bcd
Remove router binding in Meeting
d57c10b
Fix incorrect prop types
2295e1e
Split for better reading
a9e054c
Please eslint
2d263b6
Initial of SSR setup
07fedb6
Fix to universal for SSR
897be81
Add store for init state
7eed51c
Setup ssr in production
f496a43
Fix startup script
4338424
Add missing packages
0ca953d
@howawong
Copy link
Contributor

@tonyhhyip got following issues.
When running yarn start, the Web shows the following error.

Uncaught TypeError: window.gtag is not a function
    at clientEntry.js:29
    at n (createTransitionManager.js:46)
    at createTransitionManager.js:65
    at Array.forEach (<anonymous>)
    at Object.notifyListeners (createTransitionManager.js:64)
    at E (createBrowserHistory.js:78)
    at createBrowserHistory.js:169
    at Object.n [as confirmTransitionTo] (createTransitionManager.js:36)
    at Object.R [as push] (createBrowserHistory.js:149)
    at r.handleClick (Link.js:54)

and when running npm run build && npm run start
I got the following err

Refused to apply style from 'http://192.168.1.54:3000/app.8c28eaf31cb025a1d4a25456a38c18f7.css' because its MIME type ('text/html') is not a supported stylesheet MIME type, and strict MIME checking is enabled.

and the UI becomes

screen shot 2018-03-30 at 12 16 13 pm

@howawong howawong self-requested a review March 30, 2018 05:39
@howawong howawong self-assigned this Mar 30, 2018
@tonyhhyip
Copy link
Contributor Author

@howawong I have put the document, please check it

@howawong howawong merged commit 749b131 into g0vhk-io:master Mar 30, 2018
howawong added a commit that referenced this pull request Mar 30, 2018
@howawong
Enable SSR #13
8ddca7c 
- Add babel, webapck and build scripts
- Update Dockerfile
- Add server and client entries
- Add page.ejs
- Update README.md
This was referenced Mar 30, 2018
Closed
Closed
@tonyhhyip tonyhhyip mentioned this pull request Mar 31, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@howawong howawong Awaiting requested review from howawong

Assignees

@howawong howawong

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@tonyhhyip @howawong