[#88] 만들기 페이지, 보내기 페이지 API 연동 #93
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cskime
reviewed
Aug 24, 2025
src/app.jsx
Outdated
Comment on lines
12
to
20
| import { createGlobalStyle } from "styled-components"; | ||
| import NanumPenScript from "../src/assets/NanumPenScript.ttf"; | ||
|
|
||
| const GlobalStyle = createGlobalStyle` | ||
| @font-face { | ||
| font-family: 'NanumPenScript'; | ||
| src: url(${NanumPenScript}) format('truetype'); | ||
| font-weight: normal; | ||
| font-style: normal;}`; |
There was a problem hiding this comment.
- 우리 프로젝트에서는
/src/styles/global.css에 전역 스타일을 CSS로 작성하고 있는데요. - 이 코드도
global.css에 작성하거나, 아니면global.css코드를styled-components방식으로 바꿔서 방식을 통일하는게 좋을 것 같습니다. - 만약 전역 스타일을
styled-components로 만드는게 더 낫다고 생각하시면, 아래 작업을 진행해주세요./src/styles에global-style.jsx파일 생성/src/styles/global.css에 있는 내용을 포함하여GlobalStylescomponent 구현app.jsx에서global-styles.jsx를 import해서 사용index.html에서/src/styles/global.csslink 제거/src/styles/global.css파일 삭제
- 이 때, 줄 바꿈과 들여쓰기를 확인해주세요. 지금 코드에서도 들여쓰기와 줄 바꿈이 필요합니다.
There was a problem hiding this comment.
이 파일에 formatter를 실행해서 import 순서가 바뀌는 수정사항도 포함해주세요.
Comment on lines
+122
to
+125
| if (!trimmed) { | ||
| setNameError("이름을 입력해 주세요"); // 이거 안전장치로 필요할까요? | ||
| return; | ||
| } |
Comment on lines
+88
to
+93
| const imageUrls = [ | ||
| "https://picsum.photos/id/683/3840/2160", | ||
| "https://picsum.photos/id/24/3840/2160", | ||
| "https://picsum.photos/id/599/3840/2160", | ||
| "https://picsum.photos/id/1058/3840/2160", | ||
| ]; |
There was a problem hiding this comment.
- 이것도 API 연동해서 가져오도록 만들어보시면 좋겠네요.
profile-images와background-imagespath로 API를 요청하기 위해apiClient외에도baseClient라는걸 따로 만들어 두었습니다.
|
발표 시연 준비를 위해 우선 병합하겠습니다. 수정이 필요한 부분은 다른 PR을 다시 만들겠습니다. |
Closed
2 tasks
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 작업 내용
📷 스크린샷 (선택)
🧐 해결해야 하는 문제 (선택)
👀 새로 알게 된 내용 (선택)
💬 리뷰어에게 남길 말 (선택)