Skip to content

Conversation

@hyeonjin6530
Copy link
Member

@hyeonjin6530 hyeonjin6530 commented Mar 8, 2025

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 작업 내용
  • 재사용 가능한 알림창을 만들었습니다.
  • 대여 성공시 알림창을 뜨게 했습니다.
  • 대여 실패시에도 알림창을 뜨게 했습니다.

📢 To Reviewers

📸 스크린샷

현재 멘트는 대여 -> 대여 신청 으로 변경하였습니다.

대여 실패시
스크린샷 2025-03-09 오전 12 09 51

대여 성공시
스크린샷 2025-03-09 오전 12 10 22

🔗 참고 자료

@hyeonjin6530 hyeonjin6530 added the ✨ FEAT 기능 구현 label Mar 8, 2025
@hyeonjin6530 hyeonjin6530 self-assigned this Mar 8, 2025
@vercel
Copy link

vercel bot commented Mar 8, 2025

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

Name Status Preview Comments Updated (UTC)
frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 9, 2025 3:52pm

Copy link
Member

@sinji2102 sinji2102 left a comment

Choose a reason for hiding this comment

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

크게 문제는 없어 보여서 어푸햇습니당~ 고생햇어요

);
const [alertState, setAlertState] = useState<{
isOpen: boolean;
isAlertOpen: boolean;
Copy link
Member

Choose a reason for hiding this comment

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

p5) 네이밍 바꾼 거 좋습니당 더 직관적인 것 같아요!

Copy link
Member Author

Choose a reason for hiding this comment

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

감사합니다!

setErrors({});
setAlertState({ isOpen: false });
setAlertState({ isAlertOpen: false });
setMessageAlertState({ isMessageAlertOpen: false, alertMessage: '' });
Copy link
Member

Choose a reason for hiding this comment

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

p4) 네이밍이 alertState랑 messateAlert가 있어서 뭔가 헷갈리는데... AciontAlert/MessageAlert 나 Comfirm/Alert 등으로 수정하면 좋을 것 같아요!!

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 수정해서 올리도록 하겠습니당 감사합니다!

Copy link
Member

@tnals0924 tnals0924 left a comment

Choose a reason for hiding this comment

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

대여에 실패했을 때 서버에서 보내주는 에러 메시지를 모달에 띄워주시면 좋을 것 같습니다!

@hyeonjin6530
Copy link
Member Author

대여에 실패했을 때 서버에서 보내주는 에러 메시지를 모달에 띄워주시면 좋을 것 같습니다!

넵 감사합니다!!

@tnals0924 tnals0924 merged commit 8425fe5 into develop Mar 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Feat ] 물품 대여시 에러 메시지 띄우기

4 participants