[Mission5/김효리] Project_Notion_VanillaJs 과제 #53
Conversation
This file contains 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
vite, eslint, prettier, jest 적용
메인 페이지가 notionPage이고, 다른 페이지가 존재하지 않으므로 App.js에 모두 구현하도록 변경
yrnana
reviewed
Jul 14, 2023
Comment on lines
+71
to
+101
| .documents .document { | ||
| padding-left: 5px; | ||
| } | ||
|
|
||
| .documents .depth1 { | ||
| padding-left: 15px; | ||
| } | ||
|
|
||
| .documents .depth1 p { | ||
| width: 125px; | ||
| } | ||
|
|
||
| .documents .depth2 { | ||
| padding-left: 25px; | ||
| } | ||
|
|
||
| .documents .depth2 p { | ||
| width: 115px; | ||
| } | ||
|
|
||
| .documents .depth3 { | ||
| padding-left: 35px; | ||
| } | ||
|
|
||
| .documents .depth3 p { | ||
| width: 105px; | ||
| } | ||
|
|
||
| .documents .depth4 { | ||
| padding-left: 45px; | ||
| } |
yrnana
reviewed
Jul 14, 2023
src/api/api.js
Outdated
Comment on lines
3
to
4
| const request = async (url = '', options = {}) => { | ||
| const res = await fetch(`${ENV.API_END_POINT}${url}`, { |
yrnana
reviewed
Jul 14, 2023
src/domain/fetchData.js
Outdated
| ]) | ||
| return { documents, content } | ||
| default: | ||
| return null |
yrnana
reviewed
Jul 14, 2023
src/templates/mainPageTemplates.js
Outdated
| // editor | ||
| export function contentTemplate({ content, documents }) { | ||
| let html = `<textarea placeholder='새로 글을 작성해 보세요.'>${ | ||
| content ? content : '' |
yrnana
reviewed
Jul 14, 2023
src/templates/mainPageTemplates.js
Outdated
|
|
||
| function renderDocument({ id, title, documents }, depth) { | ||
| const toggleState = getItem(ENV.TOGGLE_STATE_SAVE_KEY) | ||
| const isToggled = toggleState?.[id] ? 'open' : '' |
There was a problem hiding this comment.
boolean값을 가질 것 같은 변수인데 string 값이 대응되네욤
boolean값으로 처리되고 이 변수를 사용하는 쪽에서 대응되면 좋을 것 같습니다:-)
yrnana
reviewed
Jul 14, 2023
src/utils/validation.spec.js
Outdated
|
|
||
| expect(() => { | ||
| TestComponent() | ||
| }).toThrow('TestComponent 컴포넌트 생성 시 new 연산자를 붙이지 않았습니다.') |
There was a problem hiding this comment.
에러메시지를 상수화 해서 validation에서 같이 export 하면 좋을 것 같네요ㅎㅎ
yrnana
reviewed
Jul 14, 2023
vite.config.js
Outdated
| @@ -0,0 +1,5 @@ | |||
| export default { | |||
| build: { | |||
| target: 'es2015', | |||
yrnana
reviewed
Jul 14, 2023
| <head> | ||
| <meta charset="utf-8" /> | ||
| <title>효리의 노션 클로닝 사이트</title> | ||
| <link rel="stylesheet" type="text/css" href="/styles/style.css" /> |
There was a problem hiding this comment.
스타일을 js 내에서 import 하는 방식도 고려하면 좋을 것 같습니다
- content 표현 축약 - isToggled boolean으로 변경
hyoribogo
force-pushed
the
4/#5_kimhyori_working
branch
from
aae94eb
to
41b2c52
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔗 배포 링크
📌 과제 설명
기본 요구사항
보너스 요구사항
👩💻 요구 사항과 구현 내용
시간이 많이 부족해서 기본 구현만 다 했습니다. 이후에도 계속 수정해서 업데이트 하겠습니다!
구현 완료
vite,eslint,prettier,jest등 초기 세팅을 했습니다.App.js, 페이지, 컴포넌트 파일에서 각각 상태를 전달하고 업데이트하기 위해 함수를 분리해 구현했습니다.App.js는 상태 관리를 하고, 라우팅을 처리해줍니다.MainPage.js에서는 이벤트와 업데이트 된 상태를 전달합니다.아직 구현 못한 기능들 + 추가로 하고싶은 기능들
체감 난이도 ⭐로 표시
✅ 피드백 반영사항
(추후 수정)
✅ PR 포인트 & 궁금한 점
➕ 404 페이지를 구현하기 위해 에러 전파를 마구마구 했는데, 코드가 엄청 지저분해졌습니다 ㅠ 이를 해결할 때 어떻게 해야할지 궁금합니다.