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

[FE] 방 수정 기능 추가(#616) #620

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

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 17, 2024

📓 스토리북 링크

바로가기

📌 관련 이슈

✨ PR 세부 내용

  • create, edit 으로 페이지를 나눴습니다.
  • create 페이지에 있던 코드는 RoomFormLayout으로 옮겼습니다

@github-actions github-actions bot added FE 프론트 개발 관련 작업 기능 기능 구현 작업 labels Oct 17, 2024
@chlwlstlf chlwlstlf self-assigned this Oct 17, 2024
@github-actions github-actions bot requested review from 00kang and pp449 October 17, 2024 08:47
Copy link
Contributor

@youngsu5582 youngsu5582 left a comment

Choose a reason for hiding this comment

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

☹️

}

export interface SubmitRoomInfo extends BaseRoomInfo {
id?: number;
Copy link
Contributor

Choose a reason for hiding this comment

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

id 가 없는 경우도 있나요?

Copy link
Contributor

@chlwlstlf chlwlstlf Oct 17, 2024

Choose a reason for hiding this comment

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

roomId 인데 방 생성할 땐 roomId가 필요 없습니다.
(본인이 만들어 주셨어요🤗)

@@ -12,7 +12,7 @@ const sampleRoomList = {
| "MANAGER"
| "PULL_REQUEST_NOT_SUBMITTED",
memberRole: roomInfo.memberRole as "BOTH" | "REVIEWER" | "REVIEWEE" | "NONE",
message: "FAIL시 오류 메시지",
classification: roomInfo.classification as "ALL" | "FRONTEND" | "BACKEND" | "ANDROID",
Copy link
Contributor

Choose a reason for hiding this comment

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

왜 의도적으로 as 를 사용해서 변환하는지

Copy link
Contributor

@chlwlstlf chlwlstlf Oct 17, 2024

Choose a reason for hiding this comment

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

json 데이터는 타입을 인식하지 못 해서 타입 단언해 주어야 합니다. (프론트한테 와서 왜 이러세요🤩👍)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트 개발 관련 작업 기능 기능 구현 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] 방 수정 기능 추가
2 participants