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

feat: 예약페이지 예약현황 조회 기능 #901

Merged
merged 16 commits into from
Feb 13, 2023

Conversation

yujo11
Copy link
Collaborator

@yujo11 yujo11 commented Feb 10, 2023

구현 기능

  • 예약페이지 예약현황 조회 기능
  • 맵 스크롤 시, 헤더가 같이 이동되는 버그 수정
  • 로그인 유도 팝업 문구 수정

case 별 스크린샷

회원이 본인 예약을 볼 때

image

회원이 다른 사람 예약을 볼 때

image

비회원이 비회원 예약을 볼 때

image

비회원이 회원 예약을 볼 때

image


위에 케이스 맞는지 크로스체크가 필요해서 @xrabcde 리뷰어로 추가했습니다

Close #900

@yujo11 yujo11 added 🌟 feature New feature or request 📚 frontend labels Feb 10, 2023
@yujo11 yujo11 self-assigned this Feb 10, 2023
xrabcde
xrabcde previously approved these changes Feb 10, 2023
Copy link
Collaborator

@xrabcde xrabcde left a comment

Choose a reason for hiding this comment

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

케이스별 스크린샷 확인했습니다 ~~ 로그인 케이스별 버튼유무는 전부 맞는데
아래의 경우 사용중인 예약은 수정/삭제 안되지 않나요? 저희 원래 버튼은 보이고 클릭했을 때 튕겼었나요?
확인해봐야할 것 같아욤

비회원이 비회원 예약을 볼 때

image

Comment on lines +7 to +11
export const HeaderWrapper = styled.div`
position: fixed;
width: calc(100% - 576px);
`;

Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 fixed 아니여도 되지 않나요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fiexed 없이 해결하는 방법을 찾지 못 했는데요.

다른 방법이 있다면 공유 주시면 적용해보겠습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

  • absolute -> 실패
  • 맵 영역만 따로 스크롤 -> 실패

다음에 더 생각해봅시다..

@yujo11
Copy link
Collaborator Author

yujo11 commented Feb 11, 2023

케이스별 스크린샷 확인했습니다 ~~ 로그인 케이스별 버튼유무는 전부 맞는데 아래의 경우 사용중인 예약은 수정/삭제 안되지 않나요? 저희 원래 버튼은 보이고 클릭했을 때 튕겼었나요? 확인해봐야할 것 같아욤

비회원이 비회원 예약을 볼 때

image

@xrabcde 시간이 지나간 예약에 대해 핸들링하는 처리가 빠진거 같네요

이전 예약에 대해서는 수정/삭제를 못 하도록 로직을 추가하겠습니다

@yujo11 yujo11 force-pushed the feature/900-reservation-list branch from 1003b52 to 061cf9b Compare February 11, 2023 04:29
@yujo11 yujo11 merged commit b7cd0a6 into dev Feb 13, 2023
@yujo11 yujo11 deleted the feature/900-reservation-list branch February 13, 2023 04:49
@yujo11 yujo11 mentioned this pull request Feb 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌟 feature New feature or request 📚 frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] 예약페이지 예약현황 조회 기능
4 participants