Conversation
jwoojin9
added
⛳️ team-frontend
💡 feature-admin
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
🛑 Preview Removed Reason: No commits for 7 days (stale) Push a new commit to restore the preview. |
skkuding-bot
bot
added
preview
|
✅ Syncing Preview App Succeeded Application: |
|
🛑 Preview Removed Reason: No commits for 7 days (stale) Push a new commit to restore the preview. |
|
✅ Syncing Preview App Succeeded Application: |
|
🛑 Preview Removed Reason: No commits for 7 days (stale) Push a new commit to restore the preview. |
|
❗ Syncing Preview App Failed Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
🛑 Preview Removed Reason: No commits for 7 days (stale) Push a new commit to restore the preview. |
|
❗ Syncing Preview App Failed Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
🛑 Preview Removed Reason: No commits for 7 days (stale) Push a new commit to restore the preview. |
skkuding-bot
bot
added
preview
|
❗ Syncing Preview App Failed Application: |
|
✅ Syncing Preview App Succeeded Application: |
|
이거 테스트용으로 올린 커밋인데 나중에 제대로 수정할 테니까 내일 리뷰해주세요! |
|
✅ Syncing Preview App Succeeded Application: |
|
@seoeun9 힘드시겠지만 오늘 QA 전까지 리뷰 부탁드립니다..! |
| /> | ||
| <Label htmlFor="enableMerging"> | ||
| enableMerging (obfuscation resistance) | ||
| </Label> |
There was a problem hiding this comment.
전체적으로 대기용 toast는 필요 없을 거 같아요!
모달에 로딩 중이라고 주황색 문구가 나오고 있기도 하고, 빠른 로딩의 경우 겹쳐 보이네요
그리고 토글 이름 enableMerging도 Enable Merging 이런 식으로 알아보기 쉽게 바꾸는 게 어떨까요!
| .sort((a, b) => a.order - b.order) | ||
|
|
||
| useEffect(() => { | ||
| if (problemData?.length && !selectedProblemId) { |
There was a problem hiding this comment.
problemData?.length이용해서 문제가 없는 경우의 문구도 수정해주세요! Please select a problem 이라고 나오고 있어용
| useMutation(CHECK_ASSIGNMENT_SUBMISSIONS, { | ||
| onError: (error) => { | ||
| toast.error(`Plagiarism check request failed: ${error.message}`) | ||
| setOpen(false) |
There was a problem hiding this comment.
여기도 이전 토스트에 에러가 가려집니다! 마우스를 움직여야 보여요
+에러로 종료 후에도 버튼이 checking... 에서 변하지 않는데 바꿀 방법이 있을까요
| onRequestComplete() | ||
| setOpen(false) | ||
| } else if ( | ||
| latestStatus === 'JplagError' || |
There was a problem hiding this comment.
혹시 제출물들이 wrong answer 거나.. compile error 거나.. 혹은 너무 짧으면 JplagError가 발생하나요?
정우진개론 -> 테스트 과제 쪽에서 실험해보려고 좀 끄적거렸는데 JplagError만 뜨네요.. 너무 정성이 없었나
원래 이런 건지 궁금해요!
|
@seoeun9 일단 버스에서 급하게 반영해 봤습니다! |
|
✅ Syncing Preview App Succeeded Application: |
|
❗ Syncing Preview App Failed Application: |
|
✅ Syncing Preview App Succeeded Application: |
3 participants
Description
표절검사 기능을 구현합니다
closes TAS-2510
Additional context
별도의 Figma 디자인 없이 작업한 결과물이라 Dialog 사용이라던지 text color 등 기존의 코드 컨벤션을 지키지 않은 것들이 많으므로, 현재 페이지를 바탕으로 디자인팀에서 디테일 개선안이 나오면 수정 및 리팩토링을 진행할 예정입니다.
Before submitting the PR, please make sure you do the following
fixes #123).