Skip to content

Conversation

@KYM-P
Copy link
Collaborator

@KYM-P KYM-P commented Jan 22, 2026

PR 개요

이슈 번호:

PR 체크리스트

  • Code convention을 잘 지켰나요?
  • Lint check를 수행하였나요?
  • Assignees를 추가했나요?

작업사항

  • 버그 수정
  • 신규 기능
  • 코드 스타일 수정 (포맷팅 등)
  • 리팩토링 (기능 수정 X, API 수정 X)
  • 기타

작업사항의 상세한 설명

분실물 글 작성 시에 로그인 여부와 다이얼로그를 띄우도록 수정했습니다.
기존 showLoginDialog 상태를 showFilterLoginDialog, showWriteLoginDialog 의 두 상태로 나누었습니다.
그에 따라 중복되는 코드를 줄이기 위해 LoginDialog 라는 공통 component 를 만들어 사용했습니다.

논의 사항

스크린샷

추가내용

  • develop, sprint 브랜치를 향하고 있습니다
  • production 브랜치를 향하고 있습니다

@KYM-P KYM-P requested a review from a team as a code owner January 22, 2026 16:51
@github-actions github-actions bot added campus BCSD campus ream feature labels Jan 22, 2026
@KYM-P KYM-P self-assigned this Jan 22, 2026
Copy link
Member

@kongwoojin kongwoojin left a comment

Choose a reason for hiding this comment

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

LGTM 고생하셨습니다

Comment on lines +15 to +16
val showFilterLoginDialog: Boolean = false,
val showWriteLoginDialog: Boolean = false,
Copy link
Member

Choose a reason for hiding this comment

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

나중에 Dialog State를 sealed class로 만들어도 좋을 것 같네요

@KYM-P KYM-P merged commit 574814d into feature/#1214-lostandfound-search Jan 23, 2026
4 checks passed
@KYM-P KYM-P deleted the feature/#1214-lostandfound-write-need-login branch January 23, 2026 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

campus BCSD campus ream feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants