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

[김동규] 4장: 서버 사이드 렌더링 #31

Merged
merged 1 commit into from
Jan 16, 2024
Merged

Conversation

Dongkyuuuu
Copy link
Contributor

완료!

Copy link
Member

@hyesungoh hyesungoh left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!! 👍 👍 👍

Comment on lines +9 to +12
```개인의견
Vercel에서 SSR 걷어낸 SPA용 프레임워크 한개 만들어주면 인기 많을 것 같다 라는 생각을 많이 해봤습니다..
이런 생각해보신 적 있으신가요? ㅋㅋㅋ
```
Copy link
Member

Choose a reason for hiding this comment

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

next가 export 옵션으로 빌드하게 되면 fully spa처럼 동작하긴 해요!
글애서 저는 요 방식이 필요한 경우에 즐겨 사용하긴 합니다 ㅋㅋㅋ

Comment on lines +21 to +23
```추가자료
history는 페이지 이동없이 브라우저 주소만 바꿔주는 것 이라고 보면 되고, 실제 브라우저의 뒤로가기 버튼 등을 활성화 시킨다고 합니다.
지금은 많이 안쓰지만 history 방식과 함께 많이 언급되었던 hash(https://developer.mozilla.org/en-US/docs/Web/API/Location/hash)방식도 있습니다.
Copy link
Member

Choose a reason for hiding this comment

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

👍 👍 👍

Comment on lines +85 to +87
```개인의견
개인적으로 SSR은 실제 유저가 사용하는 서비스 거나 SEO가 필요한 서비스에 사용하자는 원칙을 가지고 있습니다. 그외 나머지 프로젝트에서는 SPA를 쓰자 라는 생각입니다. 주로 백오피스 같은 프로젝트들 SPA를 사용하는 것 같아요!(서버에 들어가는 비용문제도 고려)
```
Copy link
Member

Choose a reason for hiding this comment

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

인정합니다 ... 👍

@Dongkyuuuu Dongkyuuuu merged commit 32af136 into main Jan 16, 2024
1 check passed
@Dongkyuuuu Dongkyuuuu deleted the 4장/김동규 branch January 16, 2024 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants