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.

Sign up for GitHub

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

Jump to bottom

수료증 페이지에 리더보드로 돌아가기 버튼 추가 #160

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

수료증 페이지에 리더보드로 돌아가기 버튼 추가 #160

wants to merge 3 commits into from
+46 −16

Conversation

Sunjae95
Copy link
Contributor

@Sunjae95 Sunjae95 commented Dec 26, 2024 •
edited
Loading

미리보기

image

수료증 페이지에 리더보드로 가는 링크 버튼을 추가했습니다.
링크버튼을 추가하면서 이미 방문했을 때 링크버튼 색상이 동일하게 나오지 않아 #161 여기서 수정할 예정입니다. 감안하여 확인부탁드립니다!

체크리스트

  • 이슈가 연결되어 있나요?
  • 배포 후 브라우저 콘솔에 경고나 오류가 있나요?

@Sunjae95 Sunjae95 self-assigned this Dec 26, 2024
@Sunjae95 Sunjae95 linked an issue Dec 26, 2024 that may be closed by this pull request
수료증 페이지에 리더보드로 돌아가기 버튼 추가 #159
Open
@Sunjae95 Sunjae95 marked this pull request as ready for review December 26, 2024 15:10
@Sunjae95 Sunjae95 requested a review from a team as a code owner December 26, 2024 15:10
DaleSeo
DaleSeo approved these changes Dec 26, 2024
View reviewed changes
Copy link
Contributor

@DaleSeo DaleSeo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수고하셨습니다! 추가로 발견된 이슈에 대해서 별도로 티켓 생성하니 좋네요 👍

P.S. 보드에서 티켓 가져가실 때는 본인에게 할당 좀 부탁드릴께요 🙏 이 티켓은 제가 할당해놓겠습니다.

Shot 2024-12-26 at 11 50 50

@Sunjae95
Copy link
Contributor Author

수고하셨습니다! 추가로 발견된 이슈에 대해서 별도로 티켓 생성하니 좋네요 👍

P.S. 보드에서 티켓 가져가실 때는 본인에게 할당 좀 부탁드릴께요 🙏 이 티켓은 제가 할당해놓겠습니다.

Shot 2024-12-26 at 11 50 50

감사합니다!
storybook test가 required되면서 merge를 못하는데 아마 #162 이 작업이 merge되는데까지 오래걸릴거 같지않아 merge되면 순차적으로 merge 할게요!

@Sunjae95 Sunjae95 force-pushed the 159-certificate-to-leaderboard-button branch from b653ec7 to 002ed1c Compare December 27, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DaleSeo DaleSeo DaleSeo approved these changes

Assignees

@Sunjae95 Sunjae95

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

수료증 페이지에 리더보드로 돌아가기 버튼 추가
2 participants
@Sunjae95 @DaleSeo