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.

Sign up for GitHub

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

Jump to bottom

[S4] 스튜디오 상세정보 페이지 기능 구현 완료 #190

Merged
merged 4 commits into from
Dec 27, 2024
Merged

[S4] 스튜디오 상세정보 페이지 기능 구현 완료 #190

merged 4 commits into from
Dec 27, 2024

Conversation

HuiseonDev
Copy link
Contributor

#️⃣ 연관된 이슈

#166


📝 작업 내용

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

  • 현재 접속 날짜를 영업중 여부와 비교하여 렌더링
  • 정기휴무 기능 추가
  • 공유, 북마크, 위치정보 import

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@HuiseonDev HuiseonDev added Feat 기능 추가 UI UI 작업 labels Dec 27, 2024
@HuiseonDev HuiseonDev self-assigned this Dec 27, 2024
woojoung1217
woojoung1217 approved these changes Dec 27, 2024
View reviewed changes
Copy link
Contributor

@woojoung1217 woojoung1217 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

수고하셨습니다..

@HuiseonDev
Hotfix: 옵셔널 체이닝 -> && 연산자로 수정
3edade7 
map value명 수정, key값을 Index가 아닌 Id 값으로
_id 항변환 후 적용
@woojoung1217 woojoung1217 merged commit e868068 into dev Dec 27, 2024
@HuiseonDev HuiseonDev deleted the Feat/#166 branch January 2, 2025 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kyungmim kyungmim kyungmim approved these changes

@s0zzang s0zzang s0zzang approved these changes

@aksenmi aksenmi aksenmi approved these changes

@JWJung-99 JWJung-99 JWJung-99 approved these changes

@woojoung1217 woojoung1217 woojoung1217 approved these changes

Assignees

@HuiseonDev HuiseonDev

Labels
Feat 기능 추가 UI UI 작업
Projects
Team-Flash
Status: Done
Milestone
[S4] 유저가 회원가입 및 로그인을 쉽고 빠르게 할 수 있고, 원...
Development

Successfully merging this pull request may close these issues.
6 participants
@HuiseonDev @woojoung1217 @JWJung-99 @aksenmi @s0zzang @kyungmim