Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
[Feature]: 스터디 개설 페이지 레이아웃 #21
[Feature]: 스터디 개설 페이지 레이아웃 #21
Changes from 13 commits
1ae4c49
3c8b5e3
96dcb69
10a576b
dcd3e0b
e135d97
5d35442
a77f7e6
dcbb5b3
cec09ee
333e298
f122b3d
d1b9aa6
ab0a88f
38dcff3
0ccf38f
96dead7
beae0f1
329670f
8bbc939
30d8e7c
9e03318
e09a96d
8efc686
00020f8
625326e
387950c
b963913
af73c1f
b01d21e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p3;
현재는
accessToken
여부만 파악해서 리다이렉트 시키고 있는데, 멘토/코멤 여부에 따라서 특정 경로만 허용하는 로직을 추가하면 좋을 것 같아요.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
admin 확인 여부는
isAdmin
이라는 함수를 통해서 구별할 수 있도록 했어요!0ccf38f
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
p5;
클라이언트 페이지에서는 토큰이 없으면 /auth 페이지로 가도록 해놨는데 이 부분도 논의해서 통일해두면 좋을 거 같아요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 좋습니당 🥹 env 파일 통일되어서 전역으로 불러와서 쓰면 좋겠네용.