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

스토리북 8 업그레이드 #3341

Merged
merged 9 commits into from
Aug 16, 2024
Merged

스토리북 8 업그레이드 #3341

merged 9 commits into from
Aug 16, 2024

Conversation

drakang4
Copy link
Contributor

@drakang4 drakang4 commented Aug 8, 2024

PR 설명

스토리북 8 업그레이드

변경 내역

  • 스토리북 8 업그레이드
  • msw 2 업그레이드
  • nextjs 14 업그레이드 (peer dependency는 13.4, 14 둘 다 호환됩니다.)
  • 스토리북 locale 설정 수정
  • 스토리 오류 수정

스크린샷 & URL

chromatic diff 없음

@drakang4 drakang4 added this to the v14.0.1 milestone Aug 8, 2024
@drakang4 drakang4 self-assigned this Aug 8, 2024
Copy link

codecov bot commented Aug 8, 2024

Codecov Report

Attention: Patch coverage is 0% with 24 lines in your changes missing coverage. Please review.

Project coverage is 14.77%. Comparing base (9a623a4) to head (30e8b5f).
Report is 9 commits behind head on main.

Files Patch % Lines
packages/tds-widget/src/review/mocks/reviews.ts 0.00% 24 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3341   +/-   ##
=======================================
  Coverage   14.77%   14.77%           
=======================================
  Files         721      721           
  Lines       10097    10097           
  Branches     3307     3307           
=======================================
  Hits         1492     1492           
  Misses       8182     8182           
  Partials      423      423           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment on lines +21 to +24
onContentShare: () => {},
onReviewEdit: () => {},
onScheduleAdd: () => {},
onScrapedChange: () => {},
Copy link
Contributor

Choose a reason for hiding this comment

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

요 페이지 props들이 바뀐 이유가 있을까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

스토리북 8로 오면서 visual test를 사용할 때는 preview.tsx에 argTypesRegex을 사용하기 않기를 권장해서 제거했습니다.. onXX prop을 자동으로 넣어주다가 제거되어서 이벤트 핸들러 prop의 유무로 조건 렌더링을 하는 컴포넌트의 diff가 생기지 않게 하기 위해서 추가했습니다.

Copy link
Contributor

@dongoc dongoc left a comment

Choose a reason for hiding this comment

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

👍👍

@drakang4 drakang4 force-pushed the storybook-upgrade branch from adf1d53 to 3b15773 Compare August 9, 2024 08:56
@drakang4 drakang4 enabled auto-merge (rebase) August 9, 2024 08:56
@drakang4 drakang4 disabled auto-merge August 16, 2024 04:36
@drakang4 drakang4 merged commit 8ff885e into main Aug 16, 2024
11 checks passed
@drakang4 drakang4 deleted the storybook-upgrade branch August 16, 2024 04:36
@drakang4 drakang4 mentioned this pull request Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants