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

GTB-85 [fix] 주점 관리 기능 수정 #105

Merged
merged 4 commits into from
Oct 13, 2024
Merged

GTB-85 [fix] 주점 관리 기능 수정 #105

merged 4 commits into from
Oct 13, 2024

Conversation

jwnnoh
Copy link
Member

@jwnnoh jwnnoh commented Oct 12, 2024

1. 무슨 이유로 코드를 변경했나요?

  • 주점 관리 기능을 수정하기 위함입니다.

썸네일 수정

  • 썸네일은 주점당 최대 3개까지밖에 등록을 하지 못합니다.
  • 따라서 썸네일을 비교하여 수정하기보다는, 기존 썸네일을 삭제(hard delete)하고 새로운 객체로 대체하는 방식으로 진행하였습니다.

메뉴 수정

  • 기존에 존재하는 메뉴와 새로운 메뉴를 비교하는 과정에서 수정되지 않는 기존 메뉴를 삭제하는 로직을 추가하였습니다.

2. 어떤 위험이나 장애를 발견했나요?

  • 알림톡 상세조회 시, OSIV 설정으로 인해 세션을 가지지 못하는 오류를 발견하여 이를 수정하였습니다.

3. 관련 스크린샷을 첨부해주세요.

  • 메뉴 수정 전
스크린샷 2024-10-12 오후 11 09 23
  • 메뉴 수정 이후
스크린샷 2024-10-12 오후 11 09 30

4. 완료 사항

  • 주점 관리 기능 수정

이슈 번호

close #104


5. 추가 사항

  • 알림톡 상세조회 수정

@jwnnoh jwnnoh added 🐛 bug 버그 리포트 🚑 hotfix 긴급한 코드 수정 labels Oct 12, 2024
@jwnnoh jwnnoh requested a review from rootTiket October 12, 2024 14:53
@jwnnoh jwnnoh self-assigned this Oct 12, 2024
@jwnnoh jwnnoh requested review from yechan-kim, hoonyworld and rootTiket and removed request for rootTiket, yechan-kim and hoonyworld October 12, 2024 14:53
@yechan-kim
Copy link
Contributor

LGTM 👍

@jwnnoh jwnnoh merged commit 9c47d4e into develop Oct 13, 2024
@jwnnoh jwnnoh deleted the GTB-85 branch October 13, 2024 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 버그 리포트 🚑 hotfix 긴급한 코드 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GTB-85 [fix] 주점 관리 기능 수정
2 participants