Skip to content

Conversation

@ParkGyeongTae
Copy link
Contributor

#️⃣ Issue Number

📝 요약(Summary)

  • 기존 streamlit_app.py 파일에 포함된 페이지 설정(PAGES)과 실행 로직을 분리했습니다.
  • pages_config.py 에서 페이지 구성을 전담하도록 이동하여, 관심사를 분리했습니다.
  • 코드 가독성과 유지보수성을 개선했습니다.

💬 To Reviewers (선택)

  • 페이지 정의(PAGES)를 분리한 위치(pages_config.py)가 적절한지 확인 부탁드립니다.

PR Checklist

  • 앱 실행 시 정상적으로 각 페이지로 라우팅되는지 확인
  • 페이지 추가/삭제 시 pages_config.py만 수정하면 반영되는지 확인

reference) How to Code Review

  • 따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
  • 느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
  • 물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
  • 알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.

@ParkGyeongTae ParkGyeongTae self-assigned this Oct 3, 2025
Copy link
Collaborator

@ehddnr301 ehddnr301 left a comment

Choose a reason for hiding this comment

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

👍 잘 작동합니다! 늘 구조화 관련해서 좋은 의견 내주셔서 감사합니다.

PAGES list의 첫번째로 오면 첫페이지로 설정 되네요!

image

@ParkGyeongTae ParkGyeongTae merged commit 80bb0ee into master Oct 4, 2025
2 checks passed
@ehddnr301 ehddnr301 deleted the 178-split-streamlit-main-and-pages branch October 11, 2025 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants