swsnu / swppfall2022-team4

4 stars 1 forks source link

Sprint5 PR1 #95

Closed blackruby02 closed 1 year ago

blackruby02 commented 1 year ago
blackruby02 commented 1 year ago
스크린샷 2022-11-28 오후 2 10 48 스크린샷 2022-11-28 오후 2 12 58 스크린샷 2022-11-28 오후 2 11 38 스크린샷 2022-11-28 오후 2 11 08 스크린샷 2022-11-28 오후 2 18 14

기타

blackruby02 commented 1 year ago

[가입 승인]

스크린샷 2022-11-28 오후 5 29 14(2) 스크린샷 2022-11-28 오후 5 30 02(2) 스크린샷 2022-11-28 오후 5 43 53(2)
blackruby02 commented 1 year ago

@KJYoung 님 수정사항 conflict resolve하고 동작 확인했습니다. 코드 스멜은 대부분 없앴는데 남은게 calender 쪽입니다. 지형님 코드 참고한거라 리팩토링에 시간이 좀 걸릴 것 같습니다. 다음 pr에 testing과 코드스멜 리펙토링 하겠습니다. @jihyungKo @seuha516

KJYoung commented 1 year ago

작동은 대충 확인한 것 같은데,

수정사항이 본래 의도와 부합하는지 확인해주세요 @blackruby02

blackruby02 commented 1 year ago

아 네 loop 안에 있는게 맞습니다. 감사합니다:)

KJYoung commented 1 year ago

Travis에서 eslint 및 yarn test에서 window.confirm 관련 에러가 있는 것 같습니다, 이것만 해결하면 merge해도 될 것 같아요!

jihyungKo commented 1 year ago

@KJYoung 님 수정사항 conflict resolve하고 동작 확인했습니다. 코드 스멜은 대부분 없앴는데 남은게 calender 쪽입니다. 지형님 코드 참고한거라 리팩토링에 시간이 좀 걸릴 것 같습니다. 다음 pr에 testing과 코드스멜 리펙토링 하겠습니다. @jihyungKo @seuha516

제 코드 참고하기 어려우셨을텐데.. 굉장히 수고하셨습니다!! 그런데 돌려보는 과정에서 Unexpected use of 'confirm'이라고 에러가 떠서 확인 한 번 해주실 수 있을까요? 맥이랑 윈도우 차이인지 저는 window.confirm으로 바꿨을 때 잘 돌아가네요ㅠㅠ

const deleteOnClick = () => {
    // eslint-disable-next-line no-restricted-globals
    if (confirm('삭제하시겠습니까?')) {
      if (group_id) dispatch(groupActions.deleteGroup(group_id));
    }
  };

위처럼 바꾸면 정상작동하는 것 같습니다! 위 사항과 test 약간 수정해서 커밋했습니다.

jihyungKo commented 1 year ago

++ cert 쪽에서 지금은 인증한 날짜에 색이 칠해지는게 아니라 workout log를 기록하는 날에 색이 칠해져 있는데 인증한 날에 칠해지는게 좀 더 보기 좋을 것 같아요! 이거는 천천히 수정해도 상관없을 듯 해서 이 PR 마무리하고 다음 PR에서 작업하면 좋을 것 같습니다!

sonarcloud[bot] commented 1 year ago

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

34.4% 34.4% Coverage
7.2% 7.2% Duplication