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

[HOTFIX/#38] develop 브랜치로 배포 브랜치 변경시 빌드 오류 해결 #38 #39

Merged
merged 3 commits into from
Jul 30, 2024

Conversation

clicelee
Copy link
Member

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

  • merge할 브랜치의 위치를 확인해 주세요. ( main ❌ / develop ⭕ )
  • PR의 제목에도 관련 이슈 번호를 포함시켜 주세요.
  • 리뷰가 필요한 경우 리뷰어를 지정해 주세요.
  • Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

  • Exception을 NoData로 변경하는 과정에서, 중간에 변경하지 않은 exception 변수들이 존재해 빌드 오류 발생, 해결하기
  • 파비콘 및 사이트 타이틀 잇픽으로 변경

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

Screenshot 2024-07-30 at 8 33 56 PM
Screenshot 2024-07-30 at 8 40 01 PM

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

  • Exception을 NoData로 변경하는 과정에서, 중간에 변경하지 않은 exception 변수들이 존재해 빌드 오류 발생하였고 간단한 변경으로 문제를 해결했습니다.
  • ⭐️ 이제부터 배포 브랜치는 develop입니다
  • 배포 링크를 작성하면 미리보기에 Vite+TS+ .. 이렇게 뜨는게 거슬려서 파비콘과 사이트 제목 추가하였습니다

@clicelee clicelee self-assigned this Jul 30, 2024
@clicelee clicelee requested review from yskim6772 and seojin235 July 30, 2024 11:42
Copy link

netlify bot commented Jul 30, 2024

Deploy Preview for itpick ready!

Name Link
🔨 Latest commit 5a37eb8
🔍 Latest deploy log https://app.netlify.com/sites/itpick/deploys/66a8d199d712a40008ea9eb4
😎 Deploy Preview https://deploy-preview-39--itpick.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.

return (
<div className=" w-[390px] h-[800px] mx-auto flex flex-col">
<header className="w-full flex justify-between items-center py-4">
<h1 className="text-[20px] text-black font-pretendard font-bold leading-[28px] ml-6">{nodata.title}</h1>
Copy link
Member Author

Choose a reason for hiding this comment

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

github에서 추적이 잘 안 되네요
이런 부분에서 {nodata.title}이 아니라 {exception.title} 이렇게 되어있어서 빌드 오류가 발생했고, 지금처럼 변경하여 문제 해결한것입니다

Copy link
Member

@yskim6772 yskim6772 left a comment

Choose a reason for hiding this comment

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

구웃 👍🏻👍🏻 최고 !

@clicelee clicelee changed the title [Hotfix/#38] develop 브랜치로 배포 브랜치 변경시 빌드 오류 해결 #38 [HOTFIX/#38] develop 브랜치로 배포 브랜치 변경시 빌드 오류 해결 #38 Jul 30, 2024
@clicelee clicelee merged commit ddcefcd into develop Jul 30, 2024
4 checks passed
@clicelee clicelee deleted the hotfix-clicelee/nodata-page branch July 31, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[HOTFIX] : develop 브랜치로 배포 브랜치 변경시 빌드 오류 해결
3 participants