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

[FEAT] 스와이프시 url이 변경되도록 한다. #472

Closed
1 task done
cruelladevil opened this issue Sep 27, 2023 · 0 comments · Fixed by #485
Closed
1 task done

[FEAT] 스와이프시 url이 변경되도록 한다. #472

cruelladevil opened this issue Sep 27, 2023 · 0 comments · Fixed by #485
Assignees
@cruelladevil
Labels
[ 🌞 FE ] 프론트엔드 크루들의 빛나는 개발 이야기 하나둘셋 호! ✨ Feat 꼼꼼한 기능 구현 중요하죠
Milestone
[2차 이벤트] 노래 등록 인증...

Comments

@cruelladevil
Copy link
Collaborator

cruelladevil commented Sep 27, 2023 •
edited
Loading

🤷 구현할 기능

스와이프시 url이 변경되도록 합니다.
이를 통해 링크 공유를 통해 우리 페이지 내에서 노래를 들을 수 있도록 합니다.

🔨 상세 작업 내용

  • 스와이프시 url이 변경되도록 한다.

📄 참고 사항

⏰ 예상 작업 시간

4시간

⏰ 마감일

9/29
@cruelladevil cruelladevil added [ 🌞 FE ] 프론트엔드 크루들의 빛나는 개발 이야기 하나둘셋 호! ✨ Feat 꼼꼼한 기능 구현 중요하죠 labels Sep 27, 2023
@cruelladevil cruelladevil self-assigned this Sep 27, 2023
@cruelladevil cruelladevil moved this from Todo to In Progress in 2023-shook Sep 29, 2023
@cruelladevil cruelladevil mentioned this issue Sep 29, 2023
Merged
cruelladevil added a commit that referenced this issue Sep 29, 2023
@cruelladevil
Merge branch 'main' into feat/#472
ca4a4d4
cruelladevil added a commit that referenced this issue Oct 2, 2023
@cruelladevil
Feat/#472 스와이프시 url이 변경되도록 한다. (#485)
790da5f 
* feat: 킬링파트 듣기 스와이프시 songId를 통한 url 변경

* fix: Home과 Login 링크로 가지지 않는 버그 수정

- 버그 원인은 해당 링크로 이동시 모든 refCallback(createPlayerOnObserve)이 실행됨
- YoutubeIframe에 ref로 넘겨서 임시적으로 해결하였는데 정확한 원인은 분석하지 못함

* feat: 킬링파트 공유 버튼 클릭시 현재 페이지 링크 복사

* fix: Youtube 컴포넌트에서 url을 바꾸던 것을 SongDetailItem으로 이동

- PartCollectingPage에서도 동작하는 버그 수정
- YoutubeIframe에 ref가 걸려있어서 위로 올라갈때 감지되지 않는 버그 수정

* refactor: url에 useParams와 ROUTH_PATH를 활용하도록 변경

* refactor: useValidParams로 변경

* refactor: url을 변경하는 ref 태그를 최대한 아래쪽에 배치

* refactor: preventScrollReset 삭제
@github-project-automation github-project-automation bot moved this from In Progress to Done in 2023-shook Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cruelladevil cruelladevil

Labels
[ 🌞 FE ] 프론트엔드 크루들의 빛나는 개발 이야기 하나둘셋 호! ✨ Feat 꼼꼼한 기능 구현 중요하죠
Projects
2023-shook
Status: Done
Milestone
[2차 이벤트] 노래 등록 인증샷 트워터 홍보를 준비한다.
Development

Successfully merging a pull request may close this issue.

Feat/#472 스와이프시 url이 변경되도록 한다. woowacourse-teams/2023-shook
1 participant
@cruelladevil