feat(fe): change design in duplicate course and add semester#3499
feat(fe): change design in duplicate course and add semester#3499sanghyeonjung wants to merge 2 commits intomainfrom
Conversation
|
✅ Syncing Preview App Succeeded Application: |
|
✅ Syncing Preview App Succeeded Application: |
| } | ||
|
|
||
| const classNumError = useMemo(() => { | ||
| if (classNum.trim() === ' ') { |
There was a problem hiding this comment.
이거 띄어쓰기 한칸 ' '이렇게 있는게 맞아요 아니면 '' 그냥 붙여쓰는게 맞아요? (진짜 몰라서 물어보는 거임...)
There was a problem hiding this comment.
아 이 부분은 빈칸인지 확인하는 코드인데 중현님 말씀대로 붙여 쓰는게 맞을 것 같습니다.. 수정할게요!
|
|
||
| const classNumError = useMemo(() => { | ||
| if (classNum.trim() === ' ') { | ||
| return 'Class Number must be entered.' |
There was a problem hiding this comment.
근데 이렇게 되면 맨처음 입력하라는 팝업창(?)이 뜨게 되면 아무것도 입력되지 않은 빈상태일텐데 바로 에러 나는거 아니에요?
There was a problem hiding this comment.
지금은 classNum 에 초기값이 아무것도 없어서 classNumError 가 활성화 되는 게 맞습니다. 근데, 이게 에러가 난다고 해서 터지거나 하는 게 아니라 단순하게 빨간 글씨로 경고 문구만 생기는 거라 괜찮을 것 같아요!
There was a problem hiding this comment.
근데 초기에 아무값 없는 상태에서 duplicate 버튼을 눌렀을때에만 경고가 떠야하는데, 지금 이 로직이면 classNum을 입력하는 팝업창이 뜨자마자 활성화되는거아니에요?? 아닐 수도 ㅋㅋ
There was a problem hiding this comment.
아아 근데 활성화 되는건 맞는데
이런 식으로 빨간색 글씨만 뜨는 거라 괜찮지 않을까요..??
| return '' | ||
| }, [classNum]) | ||
|
|
||
| const handleDuplicateRows = async () => { |
There was a problem hiding this comment.
지금은 에러가 있어도 복제버튼을 누르면 백엔드 서버로 요청이 날아가지네요 뭔가 방어 코드를 추가해주는 것이 좋을 것 같아요
There was a problem hiding this comment.
이 부분은 제가 에러가 있으면 복제 버튼이 비활성화 되는 식으로 변경해보도록 하겠습니다. 좋은 지적 감사합니다!
|
|
||
| console.log('selected row:', selectedRows) | ||
| console.log('selected row original:', selectedRows[0]?.original) | ||
| console.log('groupId:', selectedRows[0]?.original.id) |
There was a problem hiding this comment.
디버깅용 콘솔로그는 필요 없으면 지웁시다!
There was a problem hiding this comment.
지우는 걸 까먹었네요.. 지우겠습니다!
| } | ||
| } | ||
|
|
||
| const duplicateTarget = (id: number) => { |
There was a problem hiding this comment.
여기서 아직 id만 넘겨주고 있는데 courseNum이랑 semester랑 cLassNum 값들도 뮤테이션에 잘 실리고 있나요? 밑에서 처리해서 잘 될것 같기도!
There was a problem hiding this comment.
이 부분은 id, 어떤 코스를 복제 할 지만 외부에서 받고, 그 아래 있는, classNum, courseNum, semester 는 내부 변수에서 가져오는 거라 id만 넘겨주고 있는 거에요! 나머지 값들도 잘 넘어가고 있습니다!
There was a problem hiding this comment.
courseNum이랑 semester 입력칸에 빈값 제출하면 어떻게 되나요?
There was a problem hiding this comment.
지금은 기본 값으로 복제하려는 원본 코스의 정보를 넣어서 사용자가 일부러 빈 칸으로 만들지 않으면 그렇게 되지는 않지만, 빈 칸일 경우에 별도의 에러 메세지가 출력 되지 않습니다. 이 부분도 수정해서 빈 칸일 때 빨간 메세지가 뜨고, 복제 버튼이 비활성화되는 식으로 조치하도록 하겠습니다.
3 participants
Description
admin 에서 duplicate course 를 눌렀을 때, 입력 칸이 박스 밖으로 튀어 나오는 문제를 스크롤 바를 추가해 수정했습니다. 기존에는 input 데이터 형식을 고려하지 않고 무조건 duplicate 하도록 설계되어 있었는데, 이제 입력 형식이 맞지 않으면 빨간색 글씨가 표기 되게 변경했습니다.
Additional context
디자인이 없었던 태스크라 혹시 문제가 있으면 말해주시면 좋겠어요! 또, 이런 기능적인 내용을 추가하는 태스크가 처음이라 혹시 제대로 돌아가더라도 비효율적인 부분이나 개선하면 좋겠는 부분은 얼마든지 말씀해주시면 감사하겠습니다!
Before submitting the PR, please make sure you do the following
fixes #123).