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

[Feat/#380] 예매자(게스트) 관리하기 리뉴얼 #387

Merged
merged 49 commits into from
Aug 31, 2024
Merged

[Feat/#380] 예매자(게스트) 관리하기 리뉴얼 #387

merged 49 commits into from
Aug 31, 2024

Conversation

ocahs9
Copy link
Contributor

@ocahs9 ocahs9 commented Aug 31, 2024 •
edited
Loading

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

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

✅ Key Changes

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

  1. 작업 내용

📢 To Reviewers

  • 예매자 삭제하기는 CORS에러가 떠서 현재 정상적으로 수행되지 않습니다. (웹 - 서 slack에 남김)
  • 또한 새로고침 후 반복적으로 편집 <-> 닫기 클릭하면 데이터가 undefinded되는 에러가 발생합니다.
  • csv 추출 기능은 아직 구현되지 않았습니다. 추후 진행할 예정입니다.
  • [ Feat ] 스태프리스트, 캐스트리스트 수정 가능하도록 구현 #361 의 리뷰 내용은 해당 PR에서 차차 반영하도록 하겠습니다.

📸 스크린샷

slack에 동영상 올려두도록 하겠습니다 !
image

image

🔗 참고 자료

ocahs9 added 30 commits August 26, 2024 16:33
@ocahs9
feat: step2(스태프리스트,캐스트리스트 수정) 부활
09c961f
@ocahs9
fix: 입력 가능하도록 변경, 삭제 버튼 활성화
b755519
@ocahs9
feat: 타입 최신화
6a04442
@ocahs9
fix: 바뀐 api 명세에 맞게 타입 선언, 함수 설정
8644f2a
@ocahs9
fix: request 보낼 속성 이름으로 수정(코드 가독성 향상 목적)
2237165
@ocahs9
fix: 좀 더 가독성 좋도록 makerId로 이름 통일
f3742d7
@ocahs9
feat: 새롭게 추가된 maker(staff,cast)인지 판단하여 id 추가 여부를 결정하는 로직 구현
92b07ed
@ocahs9
fix: id가 없는 경우도 있을 것 같아서, 옵셔널로 변경
58fd1e6
@ocahs9
fix: map, every가 안되는 경우도 고려하여 ?. 연산자 사용
d403a2e
@ocahs9
fix: undefinde라 map을 사용할 수 없다는 문제 해결 - 받아오는 데이터 타입 제대로 설정
33c312d
@ocahs9
fix: 옵셔널에 .? 연산자를 안주어서 생긴 Cannot read properties of undefined 해결
e4085bf
@ocahs9
chore: 수정하기 요청 500에러 나는 이유 주석 기술
5b60d0b
@ocahs9
fix: 총 회차 수 동적으로 조절
4d2ecde
@ocahs9
fix: api로 받아온 값 읽지 못하는 버그 수정
d7fcc5f
@ocahs9
feat: 상세 이미지 타입 추가
8c66b91
@ocahs9
Merge branch 'develop' into feat/#361/ModifyManageStep2
f9c086f
@ocahs9
fix: dataState가 잘 찼는지 확인하기 위해 type 일부 옵셔녈로 변경
b2df460
@ocahs9
fix: 상세이미지 수정 가능하도록 타입 정의, 컴포넌트 추가
ed25e81
@ocahs9
fix: 잘못된 err 반환으로 인해 alert 안 뜨는 문제 수정
8ed463a
@ocahs9
fix: open-api 타입 최신화
04b8f1c
@ocahs9
fix: 에러 객체를 정상적으로 받아 사용할 수 있도록 수정
b84274e
@ocahs9
Merge branch 'develop' into feat/#361/ModifyManageStep2
e79c73f
@ocahs9
상세 이미지까지 추가해서 공연 수정하기 api 구성
73bab41
@ocahs9
fix: 공연 수정하기 조회 api에서 상세 이미지 안 불러오던 버그 해결
234d3ce
@ocahs9
fix: 공연 소개 500자로 변경
d460dc0
@ocahs9
chore: 주석 정리
7e035ec
@ocahs9
design: 자세히 보기 삭제 및 디자인 반영
12a0415
@ocahs9
fix: 10회차까지 받아들일 수 있도록 변경
11d4fc0
@ocahs9
design: 예매자 관리 - 편집 뷰 퍼블리싱
420ea1f
@ocahs9
fix: 드롭다운 필터링 10회차까지 가능하도록 수정
4d4d468
@ocahs9 ocahs9 self-assigned this Aug 31, 2024
@github-actions github-actions bot added the ✨ FEAT 기능 구현 label Aug 31, 2024
@github-actions GitHub Actions
Copy link

PR 작성하느라 고생 많았어요!! 라벨 잘 지정되었는지 확인 한 번 해 주기 🫶

pepperdad
pepperdad approved these changes Aug 31, 2024
View reviewed changes
Copy link
Member

@pepperdad pepperdad 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.

고생했어 형 ㅠㅠㅠ
정말 ,,,, QA하고 해야할 게 많겠지만 아직 너무 힘든 부분 잘 해줘서 고마웡❤️

@pepperdad
Merge branch 'develop' of https://github.com/TEAM-BEAT/BEAT-Client in…
8e71e01 
…to feat/#380/guestManageRenewal

; Conflicts:
;	src/pages/register/components/RoleWrapper.tsx
@pepperdad
Merge branch 'develop' of https://github.com/TEAM-BEAT/BEAT-Client in…
6062778 
…to feat/#380/guestManageRenewal

; Please enter a commit message to explain why this merge is necessary,
; especially if it merges an updated upstream into a topic branch.
;
; Lines starting with ';' will be ignored, and an empty message aborts
; the commit.
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
20.1% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@pepperdad pepperdad merged commit 1874e26 into develop Aug 31, 2024
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pepperdad pepperdad pepperdad approved these changes

@imddoy imddoy Awaiting requested review from imddoy

@sinji2102 sinji2102 Awaiting requested review from sinji2102

Assignees

@ocahs9 ocahs9

Labels
✨ FEAT 기능 구현
Projects
🌱beat-board
Status: 🎉Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[ Feat ] 예매자(게스트) 관리하기 리뉴얼
2 participants
@ocahs9 @pepperdad