feat(fe): enable switch between assessment problem score and testcase#3259
feat(fe): enable switch between assessment problem score and testcase#3259
Conversation
jwoojin9
added
💡 feature-admin
|
✅ Syncing Preview App Succeeded Application: |
w0uldy0u
left a comment
w0uldy0u
left a comment
There was a problem hiding this comment.
Autograded Score에서는 정상작동 하는데 FinalScore를 보여주게 해놨을때는
컬럼을 클릭해서 Testcase로 바꿔도 여전히 Score를 보여주는 문제가 있습니다
ColumnsByProblem에 반영이 안된것 같아요! |
minngyuseong
left a comment
minngyuseong
left a comment
There was a problem hiding this comment.
리뷰 마쳤습니다! props로 xxxxId로 가져오는거 싸그리 바꾸고 싶네용...
| @@ -191,17 +274,43 @@ export const createColumns = ( | |||
| }, | |||
| ...problemData.map((problem, i) => ({ | |||
There was a problem hiding this comment.
| ...problemData.map((problem, i) => ({ | |
| ...problemData.map((problem, i) => | |
| createProblemColumn(problem, i, courseId, assignmentId, isAssignmentFinished, currentView, refetch) | |
| ), |
함수로 뺴는 건 어때요..!? 너무 길어용
| label: string | ||
| }) { | ||
| const [open, setOpen] = useState(false) | ||
| const meta = column.columnDef.meta as { showAutoCounts: boolean } |
There was a problem hiding this comment.
showAutoCounts 변수명이 와닿지 않는 것 같아요. isTestcaseMode는 어때요?
| } | ||
|
|
||
| function ColumnHeaderSelector({ | ||
| column, |
There was a problem hiding this comment.
columns은 메타데이터의 showAutoCounts를 위해서 가져온 것 같습니다! 이렇게 필요한 만큼만 가져오는건 어떄요?
| column, | |
| interface ColumnHeaderSelectorProps { | |
| label: string | |
| showAutoCounts: boolean | |
| onToggle: (showAutoCounts: boolean) => void | |
| } |
다 반영하겠습니다! |
Final Score에서는 아예 작동을 안 하게 할려고 했는데 깜박했네요... 그게 맞겠죠? 어차피 testcase 개수를 보고 final score를 수동으로 매길 바애야 Auto Graded Score에서 최종 점수로 반영하는 게 편할 테니까요..? |
|
✅ Syncing Preview App Succeeded Application: |
3 participants
Description
close TAS-2243
Additional context
Before submitting the PR, please make sure you do the following
fixes #123).