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

랜딩 페이지에 파티 리스트에 드라이버 프로필 추가 #232

Merged
merged 4 commits into from
Dec 30, 2024

Conversation

rangggu
Copy link
Contributor

@rangggu rangggu commented Dec 29, 2024

✨ 작업 내용

  1. DriverBox 컴포넌트 추가
  2. 하루마다 랜덤으로 순서가 변경되도록 shuffleArray util 함수 생성

@rangggu rangggu added the ✨ Feature 기능 개발 label Dec 29, 2024
Copy link
Contributor

@DaeHee99 DaeHee99 left a comment

Choose a reason for hiding this comment

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

첫 PR 감사합니다!
고생 많으셨어요 :)

@@ -10,6 +10,8 @@ export const getRegionDriver = async (
true
);

export const getDriver = async () => await GET(`/driver/search`, true);
Copy link
Contributor

Choose a reason for hiding this comment

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

랜딩 페이지 드라이버 프로필은 모든 유저가 조회 가능해야 할 것 같아요.
즉, 로그인 여부 및 JWT 유효성 여부와 상관없이 항상 조회가 가능해야 합니다.
확인해보니, 백엔드에서 JWT가 없어도 조회가 가능하도록 api를 만들어주신 것 같아요.
그럼 해당 api를 호출할 때, JWT를 포함하지 않아도 되기 때문에 두번째 파라미터를 false로 바꿔주시면 좋을 것 같아요.
두번째 파라미터는 api 호출시 JWT를 함께 보낼지에 대한 여부입니다!

Suggested change
export const getDriver = async () => await GET(`/driver/search`, true);
export const getDriver = async () => await GET(`/driver/search`, false);

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants