-
Notifications
You must be signed in to change notification settings - Fork 0
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
base: develop
Are you sure you want to change the base?
The head ref may contain hidden characters: "feat#345/\uC5B4\uB4DC\uBBFC\uC6A9-\uD68C\uC6D0-\uC870\uD68C-api-\uC5F0\uACB0"
Conversation
…into feat#345/어드민용-회원-조회-api-연결
…into feat#345/어드민용-회원-조회-api-연결
There was a problem hiding this 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, | ||
{}, |
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
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; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
근데 0이면 설정이 안되는 것 아닝가요..?!
1. 무슨 이유로 코드를 변경했나요?
2. 어떤 위험이나 장애를 발견했나요?
3. 관련 스크린샷을 첨부해주세요.
4. 완료 사항
총 5개의 api 연결을 했습니다,, + 약간의 css 수정
5. 추가 사항
#304 pr 머지되면 axios instance를 사용하는 걸로 수정할 예정입니다.