-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/letter receive #9
base: develop
Are you sure you want to change the base?
Conversation
show list of lettes that user received.
.eslintrc.js
Outdated
'prettier/prettier': [ | ||
'error', | ||
{ | ||
endOfLine: 'auto', | ||
}, | ||
], | ||
}, | ||
}; No newline at end of file | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
뭔지 잘 모르겠음. prettierrc에서 되면 수정 부탁감사
const Wrapper = styled.div``; | ||
|
||
const Writer = styled.div``; | ||
|
||
const Content = styled.div``; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아래로 내려주세여
const Wrapper = styled.div``; | ||
|
||
const Writer = styled.div``; | ||
|
||
const Thumbnail = styled.div``; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
얘도 아래로우
<Text> | ||
<Writer>{writer}</Writer> | ||
</Text> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<Text> | |
<Writer>{writer}</Writer> | |
</Text> | |
<Writer>{writer}</Writer> |
Writer를 styled 컴포넌트로 Text 감싸게 하면 될 둣
export interface LetterProps extends Omit<LetterDataType, 'id'> { | ||
content: string; | ||
thumbnail: string; | ||
writer: string; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export interface LetterProps extends Omit<LetterDataType, 'id'> { | |
content: string; | |
thumbnail: string; | |
writer: string; | |
} | |
// eslint-disable-next-line @typescript-eslint/no-empty-interface | |
export interface LetterProps extends Omit<LetterDataType, 'id' | 'thumbnail'> {} | |
export interface LetterProps extends Omit<LetterDataType, 'content'> { | ||
id: string; | ||
thumbnail: string; | ||
writer: string; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export interface LetterProps extends Omit<LetterDataType, 'content'> { | |
id: string; | |
thumbnail: string; | |
writer: string; | |
} | |
// eslint-disable-next-line @typescript-eslint/no-empty-interface | |
export interface LetterProps extends Omit<LetterDataType, 'content'> {} |
export const emptyLetter: LetterDataType = { | ||
id: '', | ||
content: '', | ||
writer: '', | ||
thumbnail: '', | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요거 필요없을 듯
|
||
const LetterReadPage: React.FC = () => { | ||
const history = useHistory(); | ||
const letterId = history.location.pathname.substring(13); | ||
const letterToRead: LetterDataType = | ||
letters.find((letter) => letter.id === letterId) || emptyLetter; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
emptyLetter
를 안쓰는 좋은 방법을 추천드립니다!
const Wrapper = styled.div``; | ||
|
||
const ExitButton = styled.button``; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아래로오 호우!
import styled from 'styled-components'; | ||
import LetterThumbnail from '../components/Letter/LetterThumbnail'; | ||
import { letters } from '../constants/Letters'; | ||
const Wrapper = styled.div``; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아래로 메우!
No description provided.