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

group-edit.ejs 수정사항 #86

Open
youngToMaturity opened this issue Jul 30, 2021 · 2 comments
Open

group-edit.ejs 수정사항 #86

youngToMaturity opened this issue Jul 30, 2021 · 2 comments

Comments

@youngToMaturity
Copy link
Collaborator

  • 현재 group-edit.ejs에서 스터디 그룹명 입력란 옆의 수정하기 button -> 중복체크 button으로 수정하기, 기능또한 submit이후 수정이 아닌 DB상에 수정하고자 하는 그룹의 이름과 같은 이름을 가진 다른 스터디가 있는지 확인

  • 현재는 모든 결과의 redirect가 mypage이지만, 수정 완료를 제외한 모든 경우는 group-edit으로 redirect하도록

  • 참여중인 인원 아래에 스터디소개 text도 수정할 수 있도록

@youngToMaturity
Copy link
Collaborator Author

해당 issue의 해결을 위해 ...

현재의 group-edit 페이지의 대대적인 수정이 필요할 것으로 보임
why?

  • 중복체크를 위해서는 버튼 클릭시 event 발생 => fetch를 통해 확인한 후 redirect가 필요!
  • 수정 완료를 제외한 모든 경우는 group-edit으로 redirect!
    하지만, redirect를 현재의 방식으로 진행하면 redirect를 진행할때 study_group_id를 넘겨줄 수 없기 때문에 404 error가 발생합니다. 그렇기 때문에 mypage/group-edit/:study_group_id와 같이 주소를 이용하여 접근을 하는 방식이 필요할 것으로 보입니다!

@42KIM
Copy link
Collaborator

42KIM commented Aug 9, 2021

해당 issue의 해결을 위해 ...

현재의 group-edit 페이지의 대대적인 수정이 필요할 것으로 보임
why?

  • 중복체크를 위해서는 버튼 클릭시 event 발생 => fetch를 통해 확인한 후 redirect가 필요!
  • 수정 완료를 제외한 모든 경우는 group-edit으로 redirect!
    하지만, redirect를 현재의 방식으로 진행하면 redirect를 진행할때 study_group_id를 넘겨줄 수 없기 때문에 404 error가 발생합니다. 그렇기 때문에 mypage/group-edit/:study_group_id와 같이 주소를 이용하여 접근을 하는 방식이 필요할 것으로 보입니다!

생각해보니 내정보수정에서도 중복확인을 하는 부분이 있는데, 마찬가지로 수정이 필요하겠군요,..

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

No branches or pull requests

2 participants