Skip to content

Conversation

@nidor022
Copy link
Collaborator

📝 작업 내용

PR받은대로 파일명등 kebab case로 수정하였습니다.
그리고 prettier 설정 적용이 안되있어서 그거 적용하고 '' -> "" 일괄변경 되었습니다

공통 api폴더 src아래에 만들어서 그안에 axios용 파일 생성해서 인스턴스화 시켰습니다. pages아래에 MessageList.jsx파일 안에 예시용 코드 적어놓았습니다.
그외에 제가 만든 페이지 코드가 좀 섞여있긴한데 완성된게 아니라서 신경안쓰셔도됩니다 😢
@nidor022 nidor022 linked an issue Aug 12, 2025 that may be closed by this pull request
1 task
@nidor022 nidor022 added this to Rolling Aug 12, 2025
@nidor022 nidor022 moved this to Todo in Rolling Aug 12, 2025
@nidor022 nidor022 moved this from Todo to In Review in Rolling Aug 12, 2025
공통 api폴더 src아래에 만들어서 그안에 axios용 파일 생성해서 인스턴스화 시켰습니다. pages아래에 MessageList.jsx파일 안에 예시용 코드 적어놓았습니다.
@cskime cskime removed this from Rolling Aug 12, 2025
Copy link
Collaborator

@cskime cskime left a comment

Choose a reason for hiding this comment

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

굿굿

Comment on lines 3 to 7
import {
OutlinedButton,
PrimaryButton,
SecondaryButton,
} from "../components/button/button";
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 들여쓰기 해야 되지 않나요?

function StartTest() {
const [imageUrl, setImageUrl] = useState(null);

/* axios 사용용 예시코드 */
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
Collaborator

Choose a reason for hiding this comment

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

이건 지워도 되는 파일 같아요

Comment on lines 6 to 18
[ Recipient: {
id: 1111,
team: '18-3',
name: ''
backgroundColor* BackgroundColor[...]
backgroundImageURL BackgroundImageURL[...]
createdAt CreatedAt[...]
messageCount Messagecount[...]
recentMessages Recentmessages[...]
reactionCount Reactioncount[...]
topReactions Topreactions[...]
}
]
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 복사해 오시면서 자바스크립트 형식이랑 다르게 들어간거 같아요

message-list.jsx, axios-instance.js 파일이 중복되어있어서 삭제했습니다
@cskime cskime merged commit 780ecc4 into codeit-FE-18-part2:develop Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Axios 추가

4 participants