Skip to content

Conversation

@geniusYoo
Copy link
Contributor

Related issue 🛠

Work Description ✏️

permalink로 다는 것 보다 diff로 보는 게 편하실 것 같아 각 작업 사항에 대한 커밋을 링크로 달아두었습니다 :) !

  • 전체 일정 캘린더 종료날짜 정렬을 추가해 순서를 수정했어요. 솝커톤과 세미나가 같은 날이지만 세미나는 당일에 끝나고 솝커톤은 세미나 뒤 다음날에 끝나므로 종료날짜 오름차순으로 정렬했어요. 15fbb17

  • 홈 뱃지 숫자를 수정했어요. 솝탬프 랭크는 전체 파트의 솝탬프 기록을 가져와서 계산한 뒤에 해당 유저의 파트 랭크를 뽑아 리턴해야 하는데, 기존 로직은 해당 파트만을 계산 로직에 넣어 무조건 1위를 반환할 수 밖에 없었어요. (혼자만의 경쟁을 한 것임) a8411f6

    • 솝탬프는 해결했지만, fortune 관련된 것은 로컬에서는 확인하지 않았을 경우 N, 확인한 경우 빈 문자열로 응답하는 것을 확인했어요. 이것도 dev 배포 후 테스트한 뒤 결과 남기겠습니다.
  • 수료한 파트가 여러개일 경우 전부 중복 제거하지 않고 FE에 내려주도록 수정했어요.
    전부 가져오는 메서드를 작성한 뒤 7100581
    join을 이용해서 슬래시로 구분해서 FE에 내리도록 했어요. 43fbfc8

Trouble Shooting ⚽️

Related ScreenShot 📷

스웨거로

Uncompleted Tasks 😅

플그 로직이 섞여있어서, 로컬에서는 플그 로직을 임의로 처리한 뒤 테스트 완료했고 dev에 배포 후 테스트 할 예정이예요.

To Reviewers 📢

@geniusYoo geniusYoo added the 🔧 Modify 기능 수정 label Mar 26, 2025
@geniusYoo geniusYoo self-assigned this Mar 26, 2025
@height
Copy link

height bot commented Mar 26, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

@geniusYoo geniusYoo merged commit 5d1a3ff into dev Mar 26, 2025
1 check passed
Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin left a comment

Choose a reason for hiding this comment

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

시작일 정렬 후 시작일이 동일할 경우 종료일로 한번 더 정렬하니 깔끔하네요!
솝탬프 순위도 혼자만의 경쟁이었다니.. 찾느라 고생하셨겠군요ㅋㅎㅎ
파트 여러개 dev 서버로 잘 오면 영재님 qa파트 마무리 되겠네요~~ 고생하셨습니다👍

@geniusYoo geniusYoo changed the title Refactor/#508 modify with qa [Refactor] modify with qa - #508 Mar 26, 2025
@geniusYoo geniusYoo changed the title [Refactor] modify with qa - #508 [Refactor] qa 수정사항 반영 - #508 Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Refactor] QA 반영

2 participants