-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Feat/#83] 오늘의 글감 API GET #89
Conversation
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.
LGTM
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.
lgtm 배워갑니다 연결할 때 다시 찾아볼게유
@@ -11,11 +16,29 @@ interface GroupFeedAuthQueryResult { | |||
|
|||
export const useGroupFeedAuth = (groupId: string): GroupFeedAuthQueryResult => { | |||
const { data, isLoading, isError, error } = useQuery({ | |||
queryKey: ['groupFeed_Auth_moimId', groupId], | |||
queryKey: [QUERY_KEY_GROUPFEED.getGroupFeedAuth, groupId], |
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.
p5) 오 오케이 어떻게 쓰는지 확인
queryFn: () => fetchTodayWritingStyle(groupId), | ||
}); | ||
|
||
const content = data && data.data.content; |
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.
p5) 너무 깔꼼한 코드라 보기 좋아유
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.
conflict 해결하고 머지해주세요!
import Button from '../../../components/commons/Button'; | ||
|
||
interface GroupTodayWriteStylePropTypes { | ||
isMember: boolean; | ||
isMember: boolean; //나의 글 작성하기 권한 확인 | ||
groupId: string | undefined; //오늘의 주제 |
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.
[P1] 오늘의 주제인지 groupId인지 잘 모르겠는데 undefined가 올 수 있는지 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.
LGTM
} | ||
|
||
const GroupTodayWriteStyle = (props: GroupTodayWriteStylePropTypes) => { | ||
const { isMember } = props; | ||
const { isMember, groupId } = props; | ||
const { content, isLoading, isError, error } = useTodayWritingStyle(groupId || ''); |
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.
[P2] Topic으로 네이밍 가독성 좋게 해주시면 좋을 것 같아요
✨ 해당 이슈 번호 ✨
closed #83
todo
📌 내가 알게 된 부분
📌 질문할 부분
📌스크린샷(선택)