Skip to content

Feature/42 multi user환경을 위한 세션별 분리 #75

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ehddnr301
Copy link
Collaborator

#️⃣ Issue Number

📝 요약(Summary)

  • @ParkGyeongTae 님 코드에다가 추가했습니다.
  • 기존에는 모든 사용자가 하나의 공유된 graph 인스턴스를 사용하여 동시 접속 시 상태 꼬임 문제가 발생할 수 있었습니다.
  • st.session_state 를 활용해 세션마다 독립된 graph 인스턴스를 사용하도록 수정했습니다.
  • 추가로 추후 prompt 업데이트 하는것을 고려해서 graph를 새로고침 하는 버튼을 추가했습니다.

💬 To Reviewers (선택)

  • 회사환경에 적용하려면 유저별로 구분하는 것이 필요해서 나은 방법이 PR올라오기 전까진 사용하려 합니다.
    • 로그인 X (일부 사용자들로 제한해서 공개할 예정)

PR Checklist

  • lang2sql --datahub_server http://34.125.222.80:8080 run-streamlit

reference) How to Code Review

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

ParkGyeongTae and others added 2 commits April 27, 2025 00:26
추후 prompt업데이트 등에 대비하여 새로고침 버튼도 추가하였습니다.

#42, #43
Copy link
Contributor

@ParkGyeongTae ParkGyeongTae left a comment

Choose a reason for hiding this comment

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

👍 저는 로그인하지 않고 이 방식으로 임시로 사용하는거 대찬성입니당~~!

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.

2 participants