-
Notifications
You must be signed in to change notification settings - Fork 0
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
어드민 페이지 퍼블리싱 완성했습니다 #44
Conversation
간단하게 잘 만드신 것 같아요. 좋습니다! 잘못 적은 커밋 메시지의 커밋이 원격에 푸시되어도 수정해서 다시 푸시할 수 있는 방법이 있습니다. force 명령어로 강제 푸시하는 것이기에 블로그에 작성된 것처럼 위험할 수는 있겠지만 저희는 브랜치를 각자 관리하기에 큰 문제는 없을 것 같습니다. (이미 저는 미니 때 사용했었습니다만... 따로 문제 없었습니다 ㅎㅎ) |
저도 저 글을 보긴했는데 사용하지 않는게 좋다구 나와서 pr올릴때 메세지만 남겨놨습니다 |
오 벌써 만드셨네요 |
export default interface Quiz { | ||
id?: number; | ||
part: string; | ||
sectionId: number; | ||
title: string; | ||
question: string; | ||
answer: string[]; | ||
category: 'Combination' | 'MultipleChoice' | 'OXSelector' | 'ShortAnswer'; | ||
answerChoice?: 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.
카톡에서 했던 이야기 연장선인데, part와 category 를 db에서 enum으로 잡고 했어요. 요기에서는 string 카멜케이스로 req를 주는듯 한데 저는 대문자 스네이크케이스가 오는걸 전제로 작성했었습니다. 나중에 백이나 프론트에서 정한 형태로 맞춰야할듯 합니다
넵 확인했습니다
2024년 10월 10일 (목) 오전 12:48, 홍대경 ***@***.***>님이 작성:
… ***@***.**** commented on this pull request.
------------------------------
In src/admin/types/Quiz.ts
<#44 (comment)>
:
> +export default interface Quiz {
+ id?: number;
+ part: string;
+ sectionId: number;
+ title: string;
+ question: string;
+ answer: string[];
+ category: 'Combination' | 'MultipleChoice' | 'OXSelector' | 'ShortAnswer';
+ answerChoice?: string[];
+}
카톡에서 했던 이야기 연장선인데, part와 category 를 db에서 enum으로 잡고 했어요. 요기에서는 string
카멜케이스로 req를 주는듯 한데 저는 대문자 스네이크케이스가 오는걸 전제로 작성했었습니다. 나중에 백이나 프론트에서 정한 형태로
맞춰야할듯 합니다
—
Reply to this email directly, view it on GitHub
<#44 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BE2YUKYVKJDWL2C2YAGQQJTZ2VF6BAVCNFSM6AAAAABPUODGK6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNJXGU2TCNJTHE>
.
You are receiving this because you modified the open/close state.Message
ID: ***@***.***
com>
|
🔗 관련 이슈
#41
📝작업 내용
간단하게 문제 리스트랑 문제를 추가할 수 있는 어드민 페이지를 구성했습니다
기능은 백엔드 api가 나옴과 동시에 만들도록 하겠습니다.
+실수로 커밋을 잘못해서 34번 이슈에 대해 커밋을 올렸습니다.... 매우 죄송합니다
🔍 변경 사항
💬리뷰 요구사항 (선택사항)