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

[FIX] 토큰 만료 시 리다이렉트 작업, 태그 생성시 길이가 길면 알림 #888

Merged
merged 4 commits into from
Jan 1, 2025

Conversation

dmdgpdi
Copy link
Collaborator

@dmdgpdi dmdgpdi commented Dec 30, 2024

What is this PR? 🔍

Changes 📝

  • 이전엔 토큰이 만료됐을 때, 페이지에서 인터렉션 시(드래그 앤 드랍, 픽 생성, 태그 생성 등) 아무런 에러가 생성되지 않았으나 이제는 리다이렉트 되게 변경했습니다.
  • 태그 생성 및 태그 수정 시에 에러 토스트 알림 추가 및 사용자가 알지 않아도 되는 토스트 알림은 제거했습니다.

Precaution

@dmdgpdi dmdgpdi added frontend 프론트 관련 fix 🔥 bug 수정을 포함한 변경 사항 D-1 labels Dec 30, 2024
@dmdgpdi dmdgpdi requested a review from obvoso December 30, 2024 09:18
@dmdgpdi dmdgpdi self-assigned this Dec 30, 2024
Copy link
Collaborator

@sangwonsheep sangwonsheep left a comment

Choose a reason for hiding this comment

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

LGTM! 고생하셨습니다! 한가지 제안사항 드렸습니다!

Comment on lines 1 to 2
export const ERROR_MESSAGE_JSON: { [key: string]: string } = {
'AU-001': '로그인 해주세요.',
Copy link
Collaborator

Choose a reason for hiding this comment

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

p4: 백엔드 측에서 JWT 쿠키가 없거나, 변조된 쿠키인 경우 해당 에러 코드를 반환합니다.
로그인 정보가 확인되지 않았습니다. 로그인 후 다시 시도해주세요. 표현은 어떨까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵 수정했습니다!

@sangwonsheep
Copy link
Collaborator

태그의 경우 에러 토스트 알림을 보여주지 않는 방향이 더 좋아보이네요! 고생하셨습니다!

@dmdgpdi dmdgpdi merged commit 7210f9a into fe-develop Jan 1, 2025
1 check passed
@dmdgpdi dmdgpdi deleted the front/fix/#887 branch January 1, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-1 fix 🔥 bug 수정을 포함한 변경 사항 frontend 프론트 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants