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

Issue/#86 #97

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

Issue/#86 #97

wants to merge 11 commits into from

Conversation

youngToMaturity
Copy link
Collaborator

group-edit의 과정에서 발생하는 여러 문제들을 해결했습니다.
특히 그 중에서도 다른 카테고리를 변경한 후에 스터디명 중복체크를 하는 경우 새로고침에 의해 바꾼 값들이 저장되지 않는 문제를 해결했습니다.

작은 수정들

  • 수정 완료 버튼의 class가 group-create-submit이던 것을 group-edit-submit으로 수정하였습니다.
  • group-edit 페이지에 뒤로가기 버튼을 추가하였습니다. location.href = 'http://localhost:3000/mypage/group-edit';
  • location, gender의 select의 class를 수정하고, name을 삭제하였습니다. 또한, value를 영문에서 한글로 바꾸어주었습니다.
  • editForm 내부의 각 select tag가 바뀌는 경우, disabled 처리되었던 수정완료 버튼을 disabled = false;로 바꾸어주었습니다.
    따라서, 스터디 명 중복체크를 해야만 수정완료가 가능했던 issue를 해결했습니다.

큰 수정

  • URLSearchParams를 통해서 form을 submit하여 구현하던 강퇴, 중복체크, 스터디 수정의 기능을 모두 fetchthen을 활용하여 처리하도록 했습니다.
    이를 통해서 새로고침 없이 기능들이 동작하도록 했고, 강퇴 이후와 수정완료시에는 alert과 함께 mypage로 이동되도록 수정하였습니다.

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.

1 participant