swsnu / swppfall2022-team4

4 stars 1 forks source link

[FEAT] Group Refact + 고도화 #70

Closed blackruby02 closed 1 year ago

blackruby02 commented 1 year ago

<Group 고도화>

<장소>

<인증>

-All Above-

blackruby02 commented 1 year ago

group 인증에서 날짜 불러오는 부분이(달력 클릭하면 날짜 select 되는 부분) 날짜를 잘못 가져와서 front 페이지만 뺐습니다. 아마 제가 쓴게 옛날 버전이라 그런 것 같습니다. 지형님 PR 하시면 달력부분만 다시 가져오든 하면 될 것 같습니다. 나머지는 모두 되었습니다. 이 버전 pr 하고 말한 부분만 따로 파서 다시 pr 하겠습니다. front testing에서 지도 부분은 kakao를 mock 하려고 했으나 잘 안되어서 버리고 나머지 부분은 빠짐 없이 짰습니다. 카카오 맵 비중이 있는 페이지는 나머지를 다 짜도 80%가 안 넘어서 논의해보면 좋을 듯합니다.

KJYoung commented 1 year ago
blackruby02 commented 1 year ago

map 은 수정했는데 key는 잘 안되네요(key가 src로 안들어감) #78에 써놨는데 이번주 말이나 다음주에 한번더 해보겠습니다.

KJYoung commented 1 year ago

잠깐 찾아봤는데 https://theubermensch.tistory.com/229 를 참고하시면 도움이 될 것 같습니다.

blackruby02 commented 1 year ago

오.. 저걸 힌트로 요리조리 하니 되었습니다. 감사합니다! 이슈 닫겠습니다

KJYoung commented 1 year ago

Code smell 관련해서 보고 있는데, 최대 인원이 2이상일 것을 강제할 필요가 있을까요? 1인 그룹도 생성할 여지를 두는 것도 큰 문제는 없을 것 같습니다. 다른분들은 어떻게 생각하시나요? @seuha516 @jihyungKo

jihyungKo commented 1 year ago

Code smell 관련해서 보고 있는데, 최대 인원이 2이상일 것을 강제할 필요가 있을까요? 1인 그룹도 생성할 여지를 두는 것도 큰 문제는 없을 것 같습니다. 다른분들은 어떻게 생각하시나요? @seuha516 @jihyungKo

sonarcloud에서 삼항연산자같이 일부 기능 자체를 code smell로 간주하는 경우도 있어서 저는 강제하지 않아도 좋을 것 같습니다!

KJYoung commented 1 year ago
KJYoung commented 1 year ago

Notion에 frontend env 업데이트는 제가 해뒀습니다. merge된 이후에 @seuha516 님이 develop branch에서 env.tar.enc 파일 업데이트 해주시겠어요?

seuha516 commented 1 year ago

@KJYoung 넵 merge 전부 되면 한꺼번에 정리해서 push 하겠습니다

KJYoung commented 1 year ago

Merge해도 될 것 같습니다 수고하셨어요!

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 0 Code Smells

84.4% 84.4% Coverage
3.5% 3.5% Duplication