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

3 ssr fetch #16

Open
wants to merge 21 commits into
base: 2-ssr
from
Open

3 ssr fetch #16

wants to merge 21 commits into from

Conversation

@SoYoung210
Copy link
Owner

SoYoung210 commented Nov 2, 2019

Overview

client의 loadData 함수에 명시된 작업을 수행하고 초기 state를 만들어서 client에 전달합니다.
서버에서 컨텐츠가 포함된 html을 만들어서 client에 전달합니다.

FP가 느려집니다.

구성

react-router-config 를 이용해서 matching되는 route를 찾고, routes/controller 에 정의된 loadData 작업들을 수행합니다.

초기 redux state를 만들고 이 값을 이용해 html을 그려서 client에 전달합니다.
ssr-fetch

새로고침시에도 깜빡임 없이 항상 full contents를 받아오는 것을 볼 수 있습니다.

변경사항 요약

#2 와 달라진 점은, client에서 수행하던 동작을 server로 옮겼다는 것입니다.
server에서 store를 만들고 client의 loadData에 store를 전달해줌으로써, client와 server에서 비즈니스로직을 수행하는 방법이 다르지 않도록 하였습니다.

@SoYoung210 SoYoung210 force-pushed the 3-ssr-fetch branch from 913a8f0 to 639d578 Nov 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.