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: 입덕 포인트 추가, 조회 구현 #311

Merged
merged 10 commits into from
Nov 23, 2023
Merged

feat: 입덕 포인트 추가, 조회 구현 #311

merged 10 commits into from
Nov 23, 2023

Conversation

imdaxsz
Copy link
Contributor

@imdaxsz imdaxsz commented Nov 22, 2023

📝 개요

입덕 포인트 추가, 조회 구현

  • 입덕 포인트 추가
test.mp4

(현재 입덕 포인트에 DRAWING (그림체)를 포함할 경우 서버에서 500 응답이 옵니다.)

  • 리뷰 수정 시 입덕 포인트 존재 여부, 내가 남긴 입덕 포인트 조회
1.mp4

🚀 변경사항

  • 리뷰 수정 모달에서 별점, 확인 버튼에 debounce 적용

🔗 관련 이슈

#303

➕ 기타

리뷰 수정 모달 띄울 때 내가 남긴 입덕 포인트 데이터가 살짝 늦게 적용이 되는데 그렇다고 입덕 포인트 데이터 요청을 모달 밖에서 수행하기엔 마이페이지 내 리뷰 목록 같은 곳에서는 리뷰 개수만큼 요청이 될 것 같기도 하고 좀 효율적이지 않다는 생각이 들어서 일단 이렇게 뒀는데 개선할 수 있는 방법이 있을까요?🤔 로딩중일 땐 로더 컴포넌트라도 넣어야 할지 고민입니다..

+ 11/23

test.mp4

defer 적용하였습니다!


@imdaxsz imdaxsz added feature 새 기능 추가 refactor 리팩토링 labels Nov 22, 2023
@imdaxsz imdaxsz self-assigned this Nov 22, 2023
@chanwukim
Copy link
Contributor

로딩도 괜찮고 Deffer도 쓸수있을 것 같은데 UX 면에서 더 자연스러운쪽으로 고르면 좋을것같아요

key === "voiceActor" ? "VOICE_ACTOR" : key.toUpperCase(),
) as AttractionType[];

console.log("입덕 포인트: ", selectedAttraction);
Copy link
Collaborator

Choose a reason for hiding this comment

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

콘솔로그는 제거 부탁드립니다~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

요거는 입덕 포인트 수정까지 구현한 후에 제거하려고 합니다..!

Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5889ec0
Status: ✅  Deploy successful!
Preview URL: https://459652c0.oduck-client.pages.dev
Branch Preview URL: https://feature-303.oduck-client.pages.dev

View logs

@imdaxsz imdaxsz merged commit dc4ae45 into develop Nov 23, 2023
2 checks passed
@imdaxsz imdaxsz deleted the feature/303 branch November 23, 2023 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새 기능 추가 refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants