styled-component 를 v4 로 버젼업을 합니다 #262

appear · 2019-11-23T11:56:02Z

설명

styled-component 를 v4 로 버젼업을 합니다

변경 내역 및 배경

use-form-hooks 를 사용하기 위해 v4 의 ref 가 필요하다는 제이지의 말씀이 문득 떠올라서 한번 변경해보았어요
언젠가는 해야된다면 .. 지금 ... ⭐️

기존 사용하던 injectGlobal 은 v3 에서 deprecated 되었어요 createGlobal 을 사용해야되어요

사용 및 테스트 방법

DOCS

스크린샷

이 PR의 유형

버그 또는 사소한 수정
기능 추가 (하위 호환을 유지하면서 기능을 추가합니다.)
Breaking change (관련 컴포넌트를 기존에 사용하던 곳들에 코드 수정이 필요합니다.)

체크리스트

기능 추가 및 breaking change에 대한 CHANGELOG를 추가했습니다.
docs의 스토리를 변경했습니다.

boxersb · 2019-11-23T12:00:16Z

packages/core-elements/src/global-style.ts

@@ -145,3 +145,5 @@ injectGlobal`
    font-weight: bold;
  }
 `
+
+export default GlobalStyle


🤔 inject 가 아니라 그냥 만들어서 export 하는거면 주입은 _app 이나 _document 같은데서 <HEAD> 사용해서 해야하는건가욤?

아뇹 ! _app 이나 _document 에서

import GlobalStyle from 'xxx'

< GlobalStyle /> 요렇게 사용하드라구요 !

엥..? 그럼 GlobalStyle 컴포넌트??
그걸로, 페이지 감싸야하나요?

<Provider> <React.Fragment> <Navigation /> <GlobalStyle /> </React.Fragment> </Provider>

요런느낌일까여 !!

음.. 저거 있기만 하면 되는거군요..
props 던져주면 theming 도 구현할 수 있고요 ㅎㅎ

크 ... 역시 눈썰미가 대단하십니다 선배님 ! 나중에 닼모드 할떄 좋을 것 같아유

boxersb · 2019-11-25T04:29:19Z

리베이스 플리즈~

appear · 2019-11-25T04:29:34Z

packages/triple-document/package.json

@@ -18,7 +18,8 @@
    "@titicaca/core-elements": "^1.0.2",
    "@titicaca/modals": "^1.0.2",
    "@titicaca/poi-list-elements": "^1.0.2",
-    "@titicaca/react-contexts": "^1.0.2"
+    "@titicaca/react-contexts": "^1.0.2",
+    "isomorphic-fetch": "^2.2.1"


triple-document 의 coupon 에서 fetch 를 사용하고있는데 패키지 의존이 없어서 빌드타임에 터져버립니당

inbeom

Breaking change여서 triple-frontend 릴리즈 정책 잡힐 때까지 잠시만 기다려주시지요!

appear · 2019-11-25T04:36:17Z

넵 ! 급한게 아니라서용 ! 🙇

yceffort · 2019-11-25T04:37:25Z

#30 제 이슈를 드리겠습니다

appear · 2019-11-25T04:37:49Z

ㅋㅋㅋ 빠이브라닝

yceffort · 2019-11-25T04:39:37Z

ㅋㅋㅋ 빠이브라닝

앗 저건 5였네용 전 저거 4하는건줄알았는데 🤔
5로 올려야될 필요성이 생기면 제가 진행하도록 하겠습니다

polysiya

우와 이거 드디어 가는 건가요?!

appear · 2019-12-04T02:39:11Z

요거 살짝 무서워서 이욜 선생님께서

배포 해주시면 .. 확인후 내보내보겠습니다 !

inbeom · 2019-12-04T02:40:59Z

#295 리뷰와 코멘트를 부탁드립니다 🙇

boxersb · 2019-12-05T08:19:24Z

@appear 이거 리베이스 해주셔야할것 가타요..

appear · 2019-12-05T08:19:43Z

넹 이거 다음 마일스톤에 내보낼게용 ~

inbeom · 2019-12-06T06:43:37Z

요거 Rebase하고서 까나리배포로 테스트 해보실까요?!

appear · 2019-12-06T06:55:27Z

오 넵 !

inbeom · 2019-12-10T01:21:20Z

packages/core-elements/README.md

@@ -1,3 +1,3 @@
 # `@titicaca/core-elements`

-트리플의 프론트엔드 페이지에서 쓰이는 공통 디자인 요소들을 구현한 라이브러리입니다.
+트리플의 프론트엔드 페이지에서 쓰이는 공통 디자인 요소들을 구현한 라이브러리입니다.  


이거 ㅋㅋㅋㅋ 까나리를 위한 ..

엌ㅋㅋ그랬군요

appear · 2019-12-21T12:40:01Z

🙇

boxersb

고생하셨습니다~~ 항공에다 까나리 설치해볼께여~~
크리스마스 다음날 1.3.0 릴리즈시까지 묻어두시죠!!

inbeom

고고

inbeom · 2019-12-22T04:11:57Z

CHANGELOG 챙겨주십셔 🙏

appear requested a review from a team as a code owner November 23, 2019 11:56

boxersb reviewed Nov 23, 2019

View reviewed changes

appear requested a review from a team November 25, 2019 04:28

appear commented Nov 25, 2019

View reviewed changes

appear force-pushed the update-styled-v4 branch from cc3d69b to 1e2068b Compare November 25, 2019 04:30

inbeom previously approved these changes Nov 25, 2019

View reviewed changes

boxersb previously approved these changes Nov 25, 2019

View reviewed changes

inbeom requested a review from a team December 4, 2019 02:33

polysiya previously approved these changes Dec 4, 2019

View reviewed changes

inbeom assigned appear Dec 4, 2019

inbeom added common Common issue enhancement New feature or request labels Dec 4, 2019

inbeom added this to the 1.2.0 milestone Dec 4, 2019

appear removed this from the 1.2.0 milestone Dec 5, 2019

appear dismissed stale reviews from polysiya, boxersb, and inbeom via d57d9bf December 6, 2019 08:43

appear force-pushed the update-styled-v4 branch from 1e2068b to d57d9bf Compare December 6, 2019 08:43

inbeom reviewed Dec 10, 2019

View reviewed changes

giwan-dev added this to the v1.3.0 milestone Dec 12, 2019

appear added 7 commits December 20, 2019 15:04

global styled-components v4 up

4c3ea79

global style createGlobalStyle 로 변경

ea83979

peerDependencies 의 styled-component v4 up

a89cbef

label 의 color type 을 지정해줍니다

0e6f89b

triple-document 에 isomorphic-fetch dependency 를 추가합니다

ed39027

chore: core-element README.md 공백 추가

5af47ad

global-style 을 export 로 내보내도록 변경

8cf52d9

appear force-pushed the update-styled-v4 branch from 6d473ff to 8cf52d9 Compare December 20, 2019 06:05

appear requested a review from a team December 20, 2019 08:05

yceffort approved these changes Dec 21, 2019

View reviewed changes

boxersb approved these changes Dec 21, 2019

View reviewed changes

inbeom approved these changes Dec 22, 2019

View reviewed changes

appear merged commit 34dc23c into master Dec 23, 2019

delete-merged-branch bot deleted the update-styled-v4 branch December 23, 2019 00:51

inbeom mentioned this pull request Dec 26, 2019

docs에서 global style이 로드되지 않는 문제 해결 #360

Merged

5 tasks

giwan-dev mentioned this pull request Jan 3, 2020

[deps] styled-components@4 업데이트 #30

Closed

inbeom unassigned appear Sep 7, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

styled-component 를 v4 로 버젼업을 합니다 #262

styled-component 를 v4 로 버젼업을 합니다 #262

appear commented Nov 23, 2019 •

edited

Loading

boxersb Nov 23, 2019

appear Nov 23, 2019

boxersb Nov 23, 2019

appear Nov 23, 2019 •

edited

Loading

boxersb Nov 23, 2019

appear Nov 23, 2019

boxersb commented Nov 25, 2019

appear Nov 25, 2019

inbeom left a comment

appear commented Nov 25, 2019

yceffort commented Nov 25, 2019

appear commented Nov 25, 2019

yceffort commented Nov 25, 2019 •

edited

Loading

polysiya left a comment

appear commented Dec 4, 2019

inbeom commented Dec 4, 2019

boxersb commented Dec 5, 2019

appear commented Dec 5, 2019

inbeom commented Dec 6, 2019

appear commented Dec 6, 2019

inbeom Dec 10, 2019

appear Dec 10, 2019

inbeom Dec 10, 2019

appear commented Dec 21, 2019

boxersb left a comment

inbeom left a comment

inbeom commented Dec 22, 2019

styled-component 를 v4 로 버젼업을 합니다 #262

styled-component 를 v4 로 버젼업을 합니다 #262

Conversation

appear commented Nov 23, 2019 • edited Loading

설명

변경 내역 및 배경

사용 및 테스트 방법

스크린샷

이 PR의 유형

체크리스트

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

appear Nov 23, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

boxersb commented Nov 25, 2019

Choose a reason for hiding this comment

inbeom left a comment

Choose a reason for hiding this comment

appear commented Nov 25, 2019

yceffort commented Nov 25, 2019

appear commented Nov 25, 2019

yceffort commented Nov 25, 2019 • edited Loading

polysiya left a comment

Choose a reason for hiding this comment

appear commented Dec 4, 2019

inbeom commented Dec 4, 2019

boxersb commented Dec 5, 2019

appear commented Dec 5, 2019

inbeom commented Dec 6, 2019

appear commented Dec 6, 2019

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

appear commented Dec 21, 2019

boxersb left a comment

Choose a reason for hiding this comment

inbeom left a comment

Choose a reason for hiding this comment

inbeom commented Dec 22, 2019

appear commented Nov 23, 2019 •

edited

Loading

appear Nov 23, 2019 •

edited

Loading

yceffort commented Nov 25, 2019 •

edited

Loading