Skip to content

Conversation

@waldls
Copy link
Member

@waldls waldls commented Aug 25, 2025

체크리스트

  • 🧰 npm run dev로 실행 환경에서 잘 돌아가는걸 확인했나요?
  • 🎋 base 브랜치를 develop 브랜치로 설정했나요?
  • 🖌️ PR 제목은 형식에 맞게 잘 작성했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙆 리뷰어는 등록했나요?

📌 관련 이슈번호


✅ Key Changes

  • 회고 내용을 바탕으로, 제가 그동안 작업했던 파일들에서 불필요하거나 과한 주석을 제거하였습니다.
  • 백호 이용약관 연결하는 방법을 window.open -> window.location.href 로 변경하였습니다. (다른 분들의 코드와 통일성을 위함입니다.)
  • API 요청 타입을 interface -> type 으로 변경하였습니다. (다른 분들의 코드와 통일성을 위함입니다.)
  • 팀원 초대 모달의 복사 & 닫기 버튼에 cursor-pointer 속성 추가하였습니다.

📸 스크린샷 or 실행영상

페이지 동작에 변화가 있는 부분은 없습니다.


💬 To Reviewers

제거한 주석 부분 중 살리는 게 더 나은 게 있다면 알려주세요~!!

@waldls waldls added the ♻️ REFACTOR 코드 수정 및 리팩토링 label Aug 25, 2025
@github-actions
Copy link

@waldls waldls changed the title [REFACTOR] 불필요한 주석 정리 [REFACTOR] 주석 정리 및 코드 통일성 개선 Aug 25, 2025
@github-actions
Copy link

Copy link
Collaborator

@HiJuwon HiJuwon left a comment

Choose a reason for hiding this comment

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

모두 확인했습니다. 참고해서 저도 주석 정리 해보겠습니다 bb


// 워크스페이스 URL 생성
export interface CreateWorkspaceUrlRequest {
export type CreateWorkspaceUrlRequest = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

코드 통일성 향상 차원에서 type으로 맞추어주신 것도 좋습니다~~!

Copy link
Collaborator

@sunhwaaRj sunhwaaRj left a comment

Choose a reason for hiding this comment

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

링크 연결 방법과 타입정의 통일 좋아욥👍👍

@waldls waldls merged commit e1c1d91 into develop Aug 27, 2025
1 check passed
@waldls waldls deleted the feature/#207-comment-refactor branch August 27, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

♻️ REFACTOR 코드 수정 및 리팩토링 🌸 유민

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 불필요한 주석 정리

4 participants