feat: add Crescent something to StatusType of Status component

[junbong]

Summary

Status 컴포넌트에 초승달 이미지가 표시될 수 있도록 StatusType enum에 두가지 값을 추가합니다. 또한 추가 된 두가지 값일 경우와 기존에 존재하던 Lock의 경우의 렌더링 로직을 개선합니다.

Details

export enum StatusType {
  Online = 'Online',
  Offline = 'Offline',
  Lock = 'Lock',
+ OnlineCrescent = 'OnlineCrescent',
+ OfflineCrescent = 'OfflineCrescent',
}

Browser Compatibility

OS / Engine 호환성을 반드시 확인해주세요.

Windows

• [ ] Chrome - Blink
• [ ] Edge - Blink
• [ ] Firefox - Gecko (Option)

macOS

• [x] Chrome - Blink
• [ ] Edge - Blink
• [ ] Safari - WebKit
• [ ] Firefox - Gecko (Option)

References

[codecov[bot]]

Codecov Report

Merging #806 (a1bab40) into next-v1 (b1112d2) will decrease coverage by 0.01%. The diff coverage is 42.85%.

@@             Coverage Diff             @@
##           next-v1     #806      +/-   ##
===========================================
- Coverage    67.53%   67.52%   -0.02%     
===========================================
  Files          202      202              
  Lines         2831     2836       +5     
  Branches       776      778       +2     
===========================================
+ Hits          1912     1915       +3     
- Misses         802      804       +2     
  Partials       117      117              

Impacted Files	Coverage Δ
...ages/bezier-react/src/components/Status/Status.tsx	40.00% <33.33%> (+6.66%)	:arrow_up:
...ezier-react/src/components/Status/Status.styled.ts	91.66% <100.00%> (+0.75%)	:arrow_up:

---

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b1112d2...a1bab40. Read the comment docs.

[junbong]

오잉 어딜 수정해야되나요?

[quino0627]

테스트 코드를 작성한 것에 비해 전체 코드량이 늘어서 테스트 커버리지가 감소한 것으로 나오는 거 같네요? ㅋㅋㅋ 이런 경우가..

[inhibitor1217]

도메인 연관성에 관한 생각입니다!

우선순위에 관한 로직은 어플리케이션-specific한 부분이라고 생각합니다. 디자인 시스템의 인터페이스로는, status자리에 보여줄 컴포넌트는 flat하게 (예를 들면 status: 'online' | 'offline' | 'online-cresent' | 'offline-cresent' | undefined) choice를 두는 것이 좋다고 생각합니다. 방해금지와 온라인 flag에 따라 status 자리에 보여줄 icon을 결정하는 것은 어플리케이션에게 맡기면 좋겠습니다.

[sungik-choi]

우선순위에 관한 로직은 어플리케이션-specific한 부분이라고 생각합니다. 디자인 시스템의 인터페이스로는, status자리에 보여줄 컴포넌트는 flat하게 (예를 들면 status: 'online' | 'offline' | 'online-cresent' | 'offline-cresent' | undefined) choice를 두는 것이 좋다고 생각합니다. 방해금지와 온라인 flag에 따라 status 자리에 보여줄 icon을 결정하는 것은 어플리케이션에게 맡기면 좋겠습니다.

저도 동의합니다.

[junbong]

@quino0627 @sungik-choi 이거 어떻게 머지해요? 🙁

[quino0627]

우선은 그냥 해주셔도 될 것 같아요 @junbong 머지커밋으로 해주심 됩니다

[ch-builder]

:tada: This PR is included in version 1.0.0-next-v1.133 :tada:

The release is available on:

• npm package (@next-v1 dist-tag)
• GitHub release

Your semantic-release bot :package::rocket: