Skip to content

Conversation

@almighty55555
Copy link
Collaborator

#️⃣연관된 이슈

ex) #이슈번호, #이슈번호
Closes #125

📝 PR 유형

해당하는 유형에 'x'로 체크해주세요.

  • 기능 추가 (Feature)
  • 버그 수정 (Bug Fix)
  • 코드 개선 (Refactoring)
  • 스타일 변경 (UI/UX)
  • 문서 작업 (Documentation)
  • 환경 설정 (Configuration)
  • 기타 (Other)

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

가게 정보 등록

  • 배경색 회색(gray-5)
  • 최저시급 이상 확인 에러 핸들링(최저시급은 상수로 등록) -> 모달로 '시급은 최저시급 이상이어야 합니다.'
  • 1000000000원 이하 확인 에러 핸들링(최대값은 상수로 등록) -> 모달로 '시급은 1,000,000,000원 이하여야 합니다.'
  • 로그인 X => 모달 '로그인 후에 이용 가능한 기능입니다.' => /signin
  • 알바생 계정 => 모달 '사장님 계정으로만 이용 가능한 기능입니다.' => /profile
  • 가게 정보O 사장님 => /shop/edit

가게 정보 편집

  • 배경색 회색(gray-5)
  • 최저시급 이상 확인 에러 핸들링(최저시급은 상수로 등록) -> 모달로 '시급은 최저시급 이상이어야 합니다.'
  • 1000000000원 이하 확인 에러 핸들링(최대값은 상수로 등록) -> 모달로 '시급은 1,000,000,000원 이하여야 합니다.'
  • 로그인 X => 모달 '로그인 후에 이용 가능한 기능입니다.' => /signin
  • 알바생 계정 => 모달 '사장님 계정으로만 이용 가능한 기능입니다.' => /profile
  • 가게 정보X 사장님 => /shop/register

스크린샷 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@almighty55555 almighty55555 added the Bug Fix (버그 수정) 버그를 수정합니다. label May 7, 2025
@netlify
Copy link

netlify bot commented May 7, 2025

Deploy Preview for thejulge1 ready!

Name Link
🔨 Latest commit d8c84f7
🔍 Latest deploy log https://app.netlify.com/sites/thejulge1/deploys/681b847811a5e400085f3bcc
😎 Deploy Preview https://deploy-preview-138--thejulge1.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
Collaborator

@jeonghwanJay jeonghwanJay left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 ~!!

Copy link
Collaborator

@cozy-ito cozy-ito left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 👍

@almighty55555 almighty55555 merged commit 8494757 into dev May 7, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fix (버그 수정) 버그를 수정합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

가게 정보 등록, 편집 페이지 버그 픽스

4 participants