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

* Login 관련 컴포넌트들 주석 제거 및 로직 추가 요청 #205

Closed
dev-taewon-kim opened this issue Oct 15, 2021 · 0 comments · Fixed by #209
Closed

* Login 관련 컴포넌트들 주석 제거 및 로직 추가 요청 #205

dev-taewon-kim opened this issue Oct 15, 2021 · 0 comments · Fixed by #209
Assignees
Labels
API bug Something isn't working visual/ui design

Comments

@dev-taewon-kim
Copy link
Collaborator

dev-taewon-kim commented Oct 15, 2021

  • Header.js에 아직도 API URI가 localhost로 설정되어 있습니다.
  1. 필요한 Modal이라면 반드시 /api/~~ 형식으로 수정해 주세요.
  • 현재 dev에 Login Modal의 기능들이 전부 주석 처리되어 있습니다.
  1. LoginModal뿐 아니라 로그인 관련된 모든 컴포넌트의 주석 전부 제거해 주세요.
  • 추가적으로, 현재 LoginModal에서 로그인이 되어있는지 확인하는 로직이 없습니다.
  1. LoginModal, RegisterModal 등 로그인이 되어있으면 접속되면 안 되는 페이지들은 전부 로그인 여부 확인해서 로그인이 되어있으면 /로 Redirection 하도록 해주세요.
  • 반대로, Dashboard, Riskreport 등 기능 페이지에서는 로그인 여부 확인해서 로그인이 되어있지"않으면" 로그인 페이지로 Redirection 하도록 해주세요.
@dev-taewon-kim dev-taewon-kim added bug Something isn't working visual/ui design API labels Oct 15, 2021
dev-taewon-kim added a commit that referenced this issue Oct 16, 2021
@dev-taewon-kim dev-taewon-kim linked a pull request Oct 17, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API bug Something isn't working visual/ui design
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants