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

로그인하지 않아도 사용자 페이지 접근 가능하도록 RouteGuard 제거 #891

Open
wants to merge 2 commits into
base: dev/fe
Choose a base branch
from

Conversation

Hain-tain
Copy link
Contributor

@Hain-tain Hain-tain commented Oct 31, 2024

⚡️ 관련 이슈

📍주요 변경 사항

  • RouteGuard 제거

  • 참고로 존재하지 않는 멤버 아이디로 접근할 시 아래와 같이 404 페이지가 보입니다.

스크린샷 2024-10-31 오후 2 12 18

🎸기타

  • 노드모듈을 다 삭제하고 다시 깔아서인지 import 에러가 나는 부분이 있었습니다.
  • useTheme 사용하여 theme을 불러온 부분 => theme 직접 import 해와서 해결
  • SourceCode에서 ViewUpdate import => @uiw/react-codemirror 에서 불러오도록 하여 해결

🍗 PR 첫 리뷰 마감 기한

10/31

ZZZZZAP!! 오늘 안에 main 까지 올려서 재배포 하면 좋을 것 같아요!

@Hain-tain Hain-tain added bug 개발자가 의도하지 않은 상황 FE 프론트엔드 zap 리뷰 우선순위가 높은 사항 labels Oct 31, 2024
@Hain-tain Hain-tain self-assigned this Oct 31, 2024
@@ -1,6 +1,7 @@
import styled from '@emotion/styled';

import { Button } from '@/components';
import { theme } from '@/style/theme';
Copy link
Contributor

Choose a reason for hiding this comment

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

다만, 이렇게 정적 파일 변수를 import해서 쓴다면 이제 ThemeProvider를 쓸 필요가 없는 걸까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 개발자가 의도하지 않은 상황 FE 프론트엔드 zap 리뷰 우선순위가 높은 사항
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

3 participants