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

SKRF-461 fix: 비로그인시 북마크 제한 #220

Merged
merged 1 commit into from
Dec 1, 2023

Conversation

SongInjae
Copy link
Collaborator

📝요구사항과 구현내용

  • 작업 내용 1: 비로그인시 북마크 기능을 막았습니다.

구현 스크린샷

  • 없습니다.

✨pr포인트 & 궁금한 점

  • 없습니다.

@SongInjae SongInjae added the bug Something isn't working label Dec 1, 2023
@SongInjae SongInjae self-assigned this Dec 1, 2023
colorkite10
colorkite10 previously approved these changes Dec 1, 2023
@@ -59,7 +60,7 @@ const UserApplyButton = ({ eventId, eventDetail, applyModalOpen }: UserApplyButt
>
{getApplyButtonText()}
</ApplyButton>
<BookmarkButton reverse bold onClick={() => bookmarkRef.current?.click()}>
<BookmarkButton reverse bold disabled={!isToken} onClick={() => bookmarkRef.current?.click()}>
Copy link
Collaborator

Choose a reason for hiding this comment

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

참여신청하기 버튼처럼 로그인을 하지 않고 북마크 버튼을 누르면 안내 모달을 띄우면 좋을 것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

저도 하려다가 조금 걸릴 것 같아서 이렇게 했는데 딱 걸렸네요😅

hyesung99
hyesung99 previously approved these changes Dec 1, 2023
Copy link
Collaborator

@hyesung99 hyesung99 left a comment

Choose a reason for hiding this comment

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

저도 채연님과 같은 생각! 로그인으로 이동하는 모달을 보여주면 좋겠네요

@SongInjae SongInjae dismissed stale reviews from hyesung99 and colorkite10 via 5e170c2 December 1, 2023 07:37
@SongInjae SongInjae force-pushed the SKRF-461-nonlogin-bookmark-bug branch from 5e170c2 to 32017c8 Compare December 1, 2023 07:44
@SongInjae SongInjae merged commit 3df82d4 into main Dec 1, 2023
2 checks passed
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.

3 participants