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

[CHORE] 코드 정리 #101

Merged
merged 2 commits into from
Mar 1, 2024
Merged

[CHORE] 코드 정리 #101

merged 2 commits into from
Mar 1, 2024

Conversation

easyhooon
Copy link
Collaborator

@easyhooon easyhooon commented Mar 1, 2024

  • 원래 목적은 로그인 로직의 문제점을 해결하기 위해 판 브랜치였으나, 문제점을 발견하지 못함

    • intro 모듈내에 자동 로그인 관련 로직을 전에 추가해놨었는데 혹시 문제가 되는 부분이 있는지 한번 확인 부탁,,
  • 자동 로그인 실패시 로컬에 저장된 토큰 제거 로직 추가

  • 누락된 접근 제한자 추가

  • 사용하지 않는, 주석 처리된 코드 제거

  • entity @stable annotation 추가

- 누락된 접근 제한자 추가
- 사용하지 않는, 주석 처리된 코드 제거
- entity @stable 추가
@easyhooon easyhooon added the feature New feature or request label Mar 1, 2024
@easyhooon easyhooon requested a review from josushell March 1, 2024 09:17
@easyhooon easyhooon self-assigned this Mar 1, 2024
@easyhooon easyhooon merged commit dea46b9 into develop Mar 1, 2024
1 check passed
@easyhooon easyhooon deleted the fix/login branch March 1, 2024 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants