Skip to content
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

Merged
merged 123 commits into from
Oct 15, 2023
Merged

전남대_16조_애니모리_6주차 #117

merged 123 commits into from
Oct 15, 2023

Conversation

JeonDoGyun
Copy link
Contributor

구현된 기능

  • develop 브랜치로 PR 후 배포된 서버로 빌드한 결과를 확인할 수 있는 github Actions 추가했습니다.
  • 수정하기 페이지
    • 기존에 사용했던 컴포넌트와 동일한 구조에서 서버로부터 데이터를 가져와 보여주는 형태로 페이지 제작했습니다.
    • 권한이 없을 경우 보여주는 에러 페이지 간단하게 제작했습니다. -> 향후 스타일 수정 필요합니다.
  • 주변 동물 보호소 찾기 페이지
    • 지도에서 보호소 키워드 검색으로 나온 위치 마커로 표시하도록 제작했습니다.
    • 백엔드 API와 연결하여 애니모리 서비스에 등록된 보호소는 구분되게 표시하도록 만들었습니다.
  • 디렉토리 구조 변경으로 인한 README.md 수정했습니다.
  • 로그인 페이지
    • 로그인 시 Access token을 쿠키에 저장하여 사용하도록 기능 추가했습니다.
  • 등록하기 페이지
    • 라디오그룹 컴포넌트의 값이 입력되었을 때 isComplete가 제대로 동작하지 않는 문제 해결 했습니다.

hjiwon and others added 30 commits October 10, 2023 17:39
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 만들었습니다.
LimSumi and others added 27 commits October 15, 2023 01:53
fix: 등록하기 페이지 라디오 버튼 업데이트 문제 해결중
target.value 사용하여 최신값으로 validate check
fix: 라디오버튼에서 isComplete 확인하게 변경
featL 수정하기 상세 페이지 구현
내 실수로 cors 에러가 떴었음
'등록', '수정'을 props로 받아서 string concatenation하여 사용
Suspense 처럼 사용하기 위함
@LimSumi LimSumi merged commit 1a88909 into main Oct 15, 2023
1 check passed
@@ -4,7 +4,6 @@ import VMRegisterForm from './VMRegisterForm';

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])}
}, )
}

Suggested change
보고 확인해주세요...!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants