-
Notifications
You must be signed in to change notification settings - Fork 4
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
전남대_16조_애니모리_6주차 #117
전남대_16조_애니모리_6주차 #117
Conversation
style: 브랜치 테스트 위해 주석 변경
추가적으로 필요없는 주석도 삭제
폼이 전부 차있어서 isComplete일 때 오히려 validate하지 않다고 해버리는 문제가 있었습니다.
기존 겹치던 문제 해결 i Please enter the commit message for your changes. Lines starting
입력값이나 토큰에 문제가 있을 시에 isError에 잡히지 않고 data.success만 false이기 때문
생각보다 데이터가 길 수도 있음에 따라 높이와 너비 증가
feat: 등록하기 버튼 및 모달 오류 수정
GitHub Actions workflow로 npm build, jenkins deploy endpoint 호출
fix: 상세조회 페이지 잘림 현상 해결
일단 광주로 설정
feat: 현재 위치 설정 안 해도 지도 뜨도록 변경
쿠키 관리를 위해서 react-cookie 라이브러리를 package.json에 추가했습니다.
cookie.ts에 쿠키 관리할 수 있는 함수 작성했습니다. GNB에 있는 Login 버튼 쿠키 상태에 따라 변경하기 위해 getUser.ts 만들었습니다.
fix: 등록하기 페이지 라디오 버튼 업데이트 문제 해결중
target.value 사용하여 최신값으로 validate check
fix: 이메일 형식 오류 해결
fix: 라디오버튼에서 isComplete 확인하게 변경
featL 수정하기 상세 페이지 구현
내 실수로 cors 에러가 떴었음
'등록', '수정'을 props로 받아서 string concatenation하여 사용
Suspense 처럼 사용하기 위함
feat: 수정하기 페이지 완성
@@ -4,7 +4,6 @@ import VMRegisterForm from './VMRegisterForm'; | |||
|
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.
const PET_INFO_REQUIRED_KEY = [
'age',
'name',
'adoptionStatus',
'type',
tempPetInfo.weight,
tempPetInfo.description,
tempPetInfo.sex,
tempPetInfo.size,
tempPetInfo.vaccinationStatus,
tempPetInfo.neutralizationStatus,
]
const handleChange = () => {
setPetInfo(prev => {
const tempPetInfo = { ...petInfo, [fieldName]: newValue };
return {...tempPetInfo, isComplete: PET_INFO_REQUIRED_KEY.every(key => petInfo[key])}
}, )
}
보고 확인해주세요...! |
구현된 기능