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

로그아웃된 경우 자동으로 다시 로그인 #26

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

MinSeungHyun
Copy link
Owner

Fixes #20

  • 처음 설치 시 로그인 이후에 불필요하게 refresh 되지 않는지 확인
  • 자동으로 refresh되어 로그인된 직후에 한번 더 refresh되는 문제 확인 (연세포탈에서 발생하는 문제)

@MinSeungHyun MinSeungHyun added the bug Something isn't working label Dec 7, 2024
@MinSeungHyun MinSeungHyun requested a review from Copilot December 7, 2024 07:19
@MinSeungHyun MinSeungHyun self-assigned this Dec 7, 2024

Choose a reason for hiding this comment

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

Copilot reviewed 5 out of 8 changed files in this pull request and generated no suggestions.

Files not reviewed (3)
  • public/manifest.json: Language not supported
  • src/entry/background.ts: Evaluated as low risk
  • src/entry/content-scripts/learnus-login-entry-page.ts: Evaluated as low risk
Comments skipped due to low confidence (1)

src/core/login/refresh-session.ts:10

  • Ensure that the removeLastSessionRefreshedTime function is correctly defined and imported.
removeLastSessionRefreshedTime,
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

로그인 풀리는 문제
1 participant