Skip to content
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

The is_login status check seems unnecessary for the search_train function. #237

Closed
Nitro1231 opened this issue Jul 31, 2023 · 2 comments
Closed

Comments

@Nitro1231
Copy link
Contributor

우선 라이브러리 개발 노고에 감사드립니다. 덕분에 잘 사용하고 있습니다! (_ _)

SRT/SRT/srt.py

Lines 181 to 182 in 758ba02

if not self.is_login:
raise SRTNotLoggedInError()

srt.py 스크립트의 search_train 함수의 기능을 고려할때, 181~182번째 줄에 있는 is_login 확인문이 꼭 필요해 보이지 않습니다. 더 나아가 유저의 로그인 정보 없이 열차 정보를 가져오는 기능이 있다면 그냥 열차 정보만 필요할때 유용할듯 합니다. 혹시 로그인 확인문을 넣으신 이유가 있으실까요?

@Nitro1231 Nitro1231 changed the title is_login status check is not necessary for the search_train function. The is_login status check seems unnecessary for the search_train function. Jul 31, 2023
@ryanking13
Copy link
Owner

안녕하세요. 그렇네요, 검색은 굳이 로그인 체크가 필요하지 않을 것 같습니다. 관련하여 수정하는 PR을 올리는 데에 관심이 있으실까요?

@ryanking13
Copy link
Owner

Closing as resolved by #238

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

No branches or pull requests

2 participants