Skip to content

feat(badge): Badge 컴포넌트 구현 #30

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

Merged
merged 6 commits into from
Nov 27, 2024
Merged

feat(badge): Badge 컴포넌트 구현 #30

merged 6 commits into from
Nov 27, 2024

Conversation

noahluftyang
Copy link
Collaborator

변경사항

  • Badge 컴포넌트 구현
  • 사이프 홈페이지의 요소와 최대한 유사한 형태로 제작함

시각자료

화면 캡처 2024-11-26 222758

체크리스트

  • 기능 명세를 작성하였나요?
  • 테스트 코드를 작성하였나요?

추가 논의사항

Copy link

changeset-bot bot commented Nov 26, 2024

🦋 Changeset detected

Latest commit: 71bf2b1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@sipe-team/badge Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@noahluftyang noahluftyang linked an issue Nov 26, 2024 that may be closed by this pull request
@noahluftyang noahluftyang enabled auto-merge (squash) November 26, 2024 14:43
@noahluftyang noahluftyang force-pushed the feat/badge branch 2 times, most recently from c7c2ca3 to f8eb721 Compare November 27, 2024 16:30
Copy link
Contributor

@froggy1014 froggy1014 left a comment

Choose a reason for hiding this comment

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

참고해서 남은 컴포넌트 작업해볼게 -

CICD 세팅 고마워 !!

"dev:storybook": "storybook dev -p 6006",
"lint": "biome lint .",
"test": "vitest",
"typecheck": "tsc",
Copy link
Contributor

Choose a reason for hiding this comment

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

히딩크 모드 도입

coverage script도 추가해야하지 않아?

나중에 CodeCov 넣는다길래

"test:coverage": "vitest run --coverage",

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

커버리지는 모노레포 루트에서 일괄 실행하는걸로 생각 중인데, 실제 셋팅하면서 어떻게 할지 정해볼게!

Comment on lines +5 to +8
coverage: {
include: ['./src/**/*.{ts,tsx}'],
exclude: ['./src/**/*.stories.tsx', './src/env.d.ts', './src/index.ts'],
},
Copy link
Contributor

Choose a reason for hiding this comment

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

coverage 돌리니까 아무것도 실행안되길래, 찾아보니까

vitest는 glob pattern이라서 ./ 지워져야 되더라

    coverage: {
      enabled: true,
      reporter: ['text', 'json', 'html'],
      include: ['src/*.{ts,tsx}'],
      exclude: [
        'src/**/*.test.{ts,tsx}',
        'src/*.stories.{ts,tsx}',
        'src/*.d.ts',
        'src/index.ts',
      ],
      all: true,
    },
    ```

>
<Typography
asChild={true}
color="#00FFFF"
Copy link
Contributor

Choose a reason for hiding this comment

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

children에 문자열을 넣으면 반드시 teal 색상이고,

만약 이 뱃지에만 다른 색상의 문자열을 넣고 싶다면 <Element>로 넘겨줘야하는거야 ~ ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

우선은 색상 가이드가 안정해져서 teal 고정으로 구현했고 추후에 가이드 확장하면 다양한 색상 지원할 수 있을 것 같아~

@noahluftyang noahluftyang merged commit 66b81c7 into main Nov 27, 2024
3 of 4 checks passed
@noahluftyang noahluftyang deleted the feat/badge branch November 27, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Badge
2 participants