Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: 6차 QA 피드백 적용 #206

Merged
merged 16 commits into from
May 31, 2024

Conversation

kimsuyeon0916
Copy link
Contributor

@kimsuyeon0916 kimsuyeon0916 commented May 28, 2024

구현 내용/방법

  • 포트폴리오 이미지 파일 이름이 중복되는 경우와 상관없이 이미지 추가하도록 로직 변경
  • StrictMode 제거 후 새로고침 시 포트폴리오 상세 내용 안보이는 이슈 해결
  • 포트폴리오 작성/편집 폼 제출 실패했을 때도 버튼 비활성화 되는 이슈 해결
  • 포트폴리오 작성/편집 필수정보 미입력하고 폼 제출시, 필수정보 입력 모달 화면 중앙 오버레이
  • 포트폴리오 작성/편집 시 스킬, 링크 10개 초과 및 스킬 중복 시, 모달 화면 중앙 오버레이
  • 프로필 편집 폼 제출 실패했을 때도 프로필 상세 페이지로 리다이렉션 되는 이슈 해결
  • 프로필 편집 폼 제출 시, 변경이 반영 안되는 이슈 해결
  • 프로필 편집 시 스킬, 링크, 수상/활동 10개 초과 및 스킬 중복 시, 모달 화면 중앙 오버레이
  • 프로필 편집 폼 제출 중에 버튼 비활성화 적용
  • 로그인 페이지 내 이용약관 설명 제거
  • 링크 유효성 검사 UI(유효한 링크 형식) 적용

리뷰 필요

  • 이미지 파일 이름이 겹치는 경우(확장자 포함)에 파일 이름을 바꿔서 업로드 하도록 구현하였습니다! (ex. 스폰지밥.jpg -> 스폰지밥(1).jpg)
  • MuiDatepicker 유효성 검사시의 UI를 변경해보려고 했는데.. 이부분은 조금 더 걸릴 것 같습니다ㅠㅠ😢
  • 그리고 모달을 엔터/스페이스/esc로 닫는걸 구현했는데 바로 적용이 안되서 이부분을 알아보려고 합니다..!

제가 놓친 부분이 있다면 말씀 부탁드립니다!

close #202

@kimsuyeon0916 kimsuyeon0916 added Priority: Critical 우선순위: 긴급 Status: Review Needed 상태: 리뷰 대기중인 상태 Type: Bug 유형: 버그 labels May 28, 2024
@kimsuyeon0916 kimsuyeon0916 self-assigned this May 28, 2024
@prgmr99
Copy link
Member

prgmr99 commented May 28, 2024

수연님!! 작업하시느라 고생하셨습니다! 😀
작업 리스트는 모두 확인해봤는데, 잘 됩니다! 이미지 이름 변경해서 올리는 것도 되는군요!👍👍
제 환경에서만 그런지는 모르겠는데, 포트폴리오 편집할 때 스킬을 추가하거나 링크를 타이핑할 때 약간의 딜레이가 있는거 같아요! 🧐

@kimsuyeon0916
Copy link
Contributor Author

수연님!! 작업하시느라 고생하셨습니다! 😀 작업 리스트는 모두 확인해봤는데, 잘 됩니다! 이미지 이름 변경해서 올리는 것도 되는군요!👍👍 제 환경에서만 그런지는 모르겠는데, 포트폴리오 편집할 때 스킬을 추가하거나 링크를 타이핑할 때 약간의 딜레이가 있는거 같아요! 🧐

헉! 제 노트북에서는 인지를 하기 어려워서 혹시 동영상 찍어서 보내주실 수 있을 까용??!

@kimsuyeon0916 kimsuyeon0916 force-pushed the feature/#202_apply_6th_QA_issues branch from d9a91a6 to 56a4ada Compare May 31, 2024 05:57
@kimsuyeon0916 kimsuyeon0916 merged commit b1cc383 into release-1.0 May 31, 2024
@kimsuyeon0916 kimsuyeon0916 deleted the feature/#202_apply_6th_QA_issues branch November 6, 2024 04:22
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Priority: Critical 우선순위: 긴급 Status: Review Needed 상태: 리뷰 대기중인 상태 Type: Bug 유형: 버그
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: 6차 QA 피드백 적용
2 participants