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

Fix/390: [FE] [녹음] 녹음을 중단하면 녹음이 저장되지 않고 바로 삭제될 수 있도록 수정 #413

Merged
merged 3 commits into from
Feb 1, 2024
Merged

Fix/390: [FE] [녹음] 녹음을 중단하면 녹음이 저장되지 않고 바로 삭제될 수 있도록 수정 #413

merged 3 commits into from
Feb 1, 2024

Conversation

gxxrxn
Copy link
Member

@gxxrxn gxxrxn commented Jan 18, 2024

#407 PR 리뷰, 머지한 후에 확인해주세요!

구현 기능

fix: 녹음 중 삭제 기능 구현

  • 녹음 중단 시 interrupted flag 전달

fix: track player 재생 조건 수정

  • 녹음 중이거나 트랙이 있는 경우에만 재생
  • 트랙이 변경되면(녹음된 트랙이 제거되면) 트랙 재생 멈춤, 재생 시간 0으로 초기화

fix: 프로젝트 생성 녹음 상태 하단 재생바에 전달

  • 프로젝트 생성 페이지에서도 녹음 시작 시 하단 재생바의 재생 버튼이 비활성화되어야하므로 isRecording props 전달

공유하고 싶은 내용

  • 브랜치를 Fix로 만들었어야하는데 Feat으로 만들었어요.. 죄송합니당 😅

관련 이슈

@gxxrxn gxxrxn added 🐛 bug Something isn't working 📚 프론트엔드 Frontend labels Jan 18, 2024
@gxxrxn gxxrxn self-assigned this Jan 18, 2024
@vercel Vercel
Copy link

vercel bot commented Jan 18, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
collusic-new ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 1, 2024 0:52am

@gxxrxn gxxrxn linked an issue Jan 18, 2024 that may be closed by this pull request
[FE] [녹음] 녹음을 중단하면 녹음이 저장되지 않고 바로 삭제될 수 있도록 수정 #390
Open
SingTheCode
SingTheCode approved these changes Jan 23, 2024
View reviewed changes
client/src/viewmodel/UnderPlayBarViewModel.tsx Outdated
@@ -6,21 +6,31 @@ import useAudios from "hooks/useAudios";

import UnderPlayBar from "components/blocks/UnderPlayBar";

function UnderPlayBarViewModel() {
function UnderPlayBarViewModel({ isRecording }: { isRecording?: boolean }) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

파라미터에 default 값을 부여해서 타입 에러를 피할 수도 있을 것 같아요ㅎㅎ

@gxxrxn gxxrxn marked this pull request as ready for review January 24, 2024 12:29
@gxxrxn gxxrxn merged commit bd9cf8d into dev-fe Feb 1, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SingTheCode SingTheCode SingTheCode approved these changes

Assignees

@gxxrxn gxxrxn

Labels
🐛 bug Something isn't working 📚 프론트엔드 Frontend
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FE] [녹음] 녹음을 중단하면 녹음이 저장되지 않고 바로 삭제될 수 있도록 수정
2 participants
@gxxrxn @SingTheCode