Skip to content

Comments

feat: 회고 작성 UI 및 API 연동#579

Merged
JaeIn1 merged 13 commits intodevelopfrom
553-회고-작성-ui-구현
Oct 18, 2025

Hidden character warning

The head ref may contain hidden characters: "553-\ud68c\uace0-\uc791\uc131-ui-\uad6c\ud604"
Merged

feat: 회고 작성 UI 및 API 연동#579
JaeIn1 merged 13 commits intodevelopfrom
553-회고-작성-ui-구현

Conversation

@JaeIn1
Copy link
Collaborator

@JaeIn1 JaeIn1 commented Oct 16, 2025

회고 작성 UI 및 API 연동

🏄🏼‍♂️‍ Summary (요약)

  • 회고 작성하기 페이지의 UI 및 API 연동을 완료했습니다.
  • 회고 분석 페이지와 유사한 레이아웃으로, 일부 코드를 참고했습니다.

🫨 Describe your Change (변경사항)

  • 경로: /retrospect/write
  • 쿼리 파라미터: spaceId, retrospectId, title, introduction
    • 모바일과 달리 URL 쿼리로 데이터 전달
  • DesktopModal에 header에 옵셔널 속성을 추가했습니다.

🧐 전달사항

  • DesktopModal의 콘텐츠 영역에 부모 div에 패딩이 적용되어 있는데,
    회고 작성 완료 화면에서는 패딩이 다르게 적용되어 있습니다. 이 부분을 콘텐츠 내부에서 처리하는 것이 좋을지 함께 논의하고 싶습니다.

📚 Reference (참조)

2025-10-17.8.56.11.mov

@JaeIn1 JaeIn1 self-assigned this Oct 16, 2025
@JaeIn1 JaeIn1 added the 🚀 feature New feature or request label Oct 16, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 16, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • Not Rabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 553-회고-작성-ui-구현

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

1 similar comment
@coderabbitai
Copy link

coderabbitai bot commented Oct 16, 2025

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)
  • Not Rabbit

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch 553-회고-작성-ui-구현

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@prgmr99 prgmr99 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~!! LGTM👍

코드가 깔끔해서 이해하기 쉬웠슴다 :)

Comment on lines +33 to +35
font-size: ${isDesktop ? "1.6rem" : "1.4rem"};
font-weight: ${isDesktop ? "600" : "500"};
line-height: ${isDesktop && "140%"};
Copy link
Member

Choose a reason for hiding this comment

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

contentsTypography를 적용하기에는 쉽지 않을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

엇 제가 의도를 정확히 이해하지 못한 것 같습니다..

Copy link
Member

Choose a reason for hiding this comment

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

하단에 contents 부분에 반영이 되는 스타일링 같아서요..!
아래와 같이 적용해도 되지 않을까해서 남겨봤습니다!

<Typography variant={isDesktop ? ~~ : ~~} >{contents}</Typography>

Copy link
Member

@klmhyeonwoo klmhyeonwoo Oct 18, 2025

Choose a reason for hiding this comment

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

승준님이 추천해주신 부분 말고도
${isDekstop ? css``" : "css``} 형태로도 모바일과 데스크탑 스타일의 구분이 하나의 분기로 명확하게 나눠질 수 있을 것 같아요!

introduction: string;
};

export const AdvanceQuestionsNum = 2;
Copy link
Member

Choose a reason for hiding this comment

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

혹시 요거는 무슨 변수일까요??!

Copy link
Collaborator Author

@JaeIn1 JaeIn1 Oct 17, 2025

Choose a reason for hiding this comment

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

진행사항, 목표 진행률에 합계를 나타내는 변수입니다!!

@JaeIn1 JaeIn1 changed the title feat: 회고 작성 UI 구현 feat: 회고 작성 UI 및 API 연동 Oct 17, 2025
type?: "confirm" | "alert";
buttonText?: string[];
autoClose?: boolean;
enableHeader?: boolean;
Copy link
Member

Choose a reason for hiding this comment

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

이게 그 데스크탑 모달의 헤더를 지우는 옵셔널로 추가된 부분 맞죠?!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

넵넵 맞습니다!!

Copy link
Member

@klmhyeonwoo klmhyeonwoo left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 작업량이 많네요 ;)

@JaeIn1 JaeIn1 merged commit ecafcf1 into develop Oct 18, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚀 feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants