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

#345 Feat: 사용자 어드민 api 연결 #355

Open
wants to merge 15 commits into
base: develop
Choose a base branch
from

Conversation

dalzzy
Copy link
Member

@dalzzy dalzzy commented Feb 12, 2025

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

image


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


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

image
image
image
image
image


4. 완료 사항

총 5개의 api 연결을 했습니다,, + 약간의 css 수정

5. 추가 사항

#304 pr 머지되면 axios instance를 사용하는 걸로 수정할 예정입니다.

@dalzzy dalzzy added Feat 기능 개발 Refactor refactoring labels Feb 12, 2025
@dalzzy dalzzy requested a review from woneeeee February 12, 2025 14:10
@dalzzy dalzzy self-assigned this Feb 12, 2025
@dalzzy dalzzy linked an issue Feb 12, 2025 that may be closed by this pull request
Copy link
Collaborator

@woneeeee woneeeee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!! 다른 api들도 파이팅..!!

const url = `${BASE_URL}${PATH}${endpoint}?${params}`;
const response = await axios.patch(
url,
{},
Copy link
Collaborator

Choose a reason for hiding this comment

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

쉽지 않았던... 401이슈 고생하셧습니다...👍🏻


export type MemberData = {
id: number;
Copy link
Collaborator

Choose a reason for hiding this comment

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

타입 정의가 진짜 빡세네요...

background-color: #ffffff;
border: 1px solid #f2f2f2;
box-shadow: 0px 3px 8px 0px rgba(133, 141, 138, 0.2);
flex-shrink: 0;
Copy link
Collaborator

Choose a reason for hiding this comment

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

근데 0이면 설정이 안되는 것 아닝가요..?!

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

Successfully merging this pull request may close these issues.

feat: 어드민용 회원 조회 API 연결
2 participants