Skip to content

Conversation

@strfunctionk
Copy link
Contributor

#️⃣ Related Issues

연관된 모든 Issue를 작성해주세요.

🧑‍💻 작업 내용

어떤 작업을 진행했는지 자세하게 작성해주세요.

  • sns 이벤트 어시스턴트 및 프로필 모달 fallback 추가 및 disable 추가

💾 작업 결과 (선택)

사진, 영상 등을 첨부해주세요.

💬 리뷰 시 요청사항 (선택)

Reviewer는 코드 리뷰의 코멘트에 코멘트를 강조하고 싶은 정도를 Pn 규칙
맞춰서 표기해 주세요.

📝 Checklist

  • Reviewer를 추가했나요?
  • Convention을 준수했나요?

@strfunctionk strfunctionk self-assigned this Aug 21, 2025
Copilot AI review requested due to automatic review settings August 21, 2025 19:42
@strfunctionk strfunctionk requested a review from a team as a code owner August 21, 2025 19:42
@strfunctionk strfunctionk requested review from S-Gihun and kyeoungwoon and removed request for a team August 21, 2025 19:42
@strfunctionk strfunctionk added the 🔧 Fix 기능 수정사용자가 동작했을 때 잘못된 동작을 정상 동작으로 변경 예: 버튼이 동작하지 않던 문제 해결 label Aug 21, 2025
@strfunctionk strfunctionk requested a review from duwlsssss August 21, 2025 19:42
@strfunctionk strfunctionk linked an issue Aug 21, 2025 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Aug 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
haru-web Ready Ready Preview Comment Aug 21, 2025 7:43pm

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

SNS 이벤트 어시스턴트와 프로필 설정 모달에 대한 fallback 페이지를 추가하고, 멤버 초대 시 loading 상태를 관리하는 기능을 구현했습니다.

  • SNS 이벤트 어시스턴트 모달들에 대한 독립 실행 페이지들을 추가
  • 프로필 설정 관련 모달들의 독립 실행 페이지 구조 개선
  • 멤버 초대 시 loading 상태 표시 기능 추가

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
WorkspaceSettingsMenu.client.tsx 멤버 초대 시 loading 상태를 추적하고 컴포넌트에 전달
settings/page.tsx 설정 모달의 독립 실행 페이지를 Suspense와 함께 구현
settings/change-password/page.tsx 비밀번호 변경 모달의 독립 실행 페이지를 Suspense와 함께 구현
new-event/page.tsx 새 이벤트 생성 모달의 독립 실행 페이지 구현
delete/page.tsx 이벤트 삭제 모달의 독립 실행 페이지 구현
creating-event/page.tsx 이벤트 생성 중 모달의 독립 실행 페이지 구현
download/page.tsx 이벤트 다운로드 모달의 독립 실행 페이지 구현
[...catchAll]/page.tsx 처리되지 않은 라우트들에 대한 catch-all 페이지 추가

/>
</>
);
}
Copy link

Copilot AI Aug 21, 2025

Choose a reason for hiding this comment

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

[nitpick] [P3] 함수 컴포넌트가 세미콜론으로 끝나지 않습니다. 일관성을 위해 함수 표현식 뒤에 세미콜론을 추가해주세요.

Suggested change
}
};

Copilot uses AI. Check for mistakes.
/>
</>
);
}
Copy link

Copilot AI Aug 21, 2025

Choose a reason for hiding this comment

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

[nitpick] [P3] 함수 컴포넌트가 세미콜론으로 끝나지 않습니다. 일관성을 위해 함수 표현식 뒤에 세미콜론을 추가해주세요.

Suggested change
}
};

Copilot uses AI. Check for mistakes.
/>
</>
);
}
Copy link

Copilot AI Aug 21, 2025

Choose a reason for hiding this comment

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

[nitpick] [P3] 함수 컴포넌트가 세미콜론으로 끝나지 않습니다. 일관성을 위해 함수 표현식 뒤에 세미콜론을 추가해주세요.

Suggested change
}
};

Copilot uses AI. Check for mistakes.
<MainWithSnsDetailPage />
</>
);
}
Copy link

Copilot AI Aug 21, 2025

Choose a reason for hiding this comment

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

[nitpick] [P3] 함수 컴포넌트가 세미콜론으로 끝나지 않습니다. 일관성을 위해 함수 표현식 뒤에 세미콜론을 추가해주세요.

Suggested change
}
};

Copilot uses AI. Check for mistakes.
@S-Gihun S-Gihun merged commit 397088e into develop Aug 22, 2025
2 of 3 checks passed
@S-Gihun S-Gihun deleted the fix/#394-sns-event-and-profile branch August 22, 2025 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 Fix 기능 수정사용자가 동작했을 때 잘못된 동작을 정상 동작으로 변경 예: 버튼이 동작하지 않던 문제 해결

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] SNS 이벤트 어시스턴트 및 프로필 수정

4 participants