Skip to content

Conversation

@ohprettyhak
Copy link
Contributor

🌍 이슈 번호

✅ 작업 내용

  • 게임 기본 정보 수정 페이지에서 일부 데이터가 잘못 설정되어 저장되지 않는 오류를 수정했습니다.
    • videoIdnull일 경우, 유튜브 prefix가 적용되지 않도록 수정했습니다.
    • 유튜브 링크가 �유효하지 않는 경우(videoId에 입력 존재) 텍스트를 출력하도록 변경했습니다.
    • 오류로 인해 저장이 이루어지지 않은 경우 대화 상자가 표시되도록 구현했습니다.

@ohprettyhak ohprettyhak added the fix Bug fixes label Mar 26, 2024
@ohprettyhak ohprettyhak self-assigned this Mar 26, 2024
@ohprettyhak ohprettyhak linked an issue Mar 26, 2024 that may be closed by this pull request
3 tasks
@netlify
Copy link

netlify bot commented Mar 26, 2024

Deploy Preview for hufscheer ready!

Name Link
🔨 Latest commit c4168ff
🔍 Latest deploy log https://app.netlify.com/sites/hufscheer/deploys/6602bf53909f7e0008a2e30d
😎 Deploy Preview https://deploy-preview-199--hufscheer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 42
Accessibility: 71
Best Practices: 83
SEO: 55
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Mar 26, 2024

Deploy Preview for hufscheer-manager ready!

Name Link
🔨 Latest commit c4168ff
🔍 Latest deploy log https://app.netlify.com/sites/hufscheer-manager/deploys/6602bf538f2ab80008a304fe
😎 Deploy Preview https://deploy-preview-199--hufscheer-manager.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Member

@seongminn seongminn left a comment

Choose a reason for hiding this comment

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

헉 제가 실수한 부분인 것 같은데.. 감사합니다🥲👍

@ohprettyhak ohprettyhak merged commit 0bf5c30 into main Mar 26, 2024
@ohprettyhak ohprettyhak deleted the fix/manager/game-edit branch March 26, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 매니저 게임 기본 정보 저장 오류 해결

3 participants