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

등록하기, 수정하기 페이지 모달 안내 텍스트 요청에 따라 적절하게 변경 #174

Merged
merged 4 commits into from
Nov 8, 2023

Conversation

JeonDoGyun
Copy link
Contributor

구현된 기능

  • 비디오 화면과 유저 정보 드롭박스가 겹쳐서 click이 되지 않는 문제 해결
  • 모달 동작 방식에 따른 안내텍스트와 버튼에 변화가 생기도록 변경
  • any 타입, console.log()로 생긴 warning 최대한 고쳤습니다.

미구현된 기능

  • HeaderModal에 있는 data는 아직 any 타입으로 되어있습니다.
  • 모달에 들어가는 텍스트의 경우, string, Array<string>을 useState로 관리하고 있는데, 이 부분은 custom hook으로 만들면 리팩토링이 가능할 것 같아서 수정하겠습니다.

z index를 바꿔서 clickable하게 했습니다.
401, 403, 404 등 status code에 따라 button 개수와 내용이 다르게 모달에 나오도록 했습니다.
등록하기 -> 예 버튼을 누르면 아무 액션이 없는 것에서 안내텍스트와 돌아가기 버튼이 나오도록 구현했습니다.
추가로 warning 표시 나오는 any 타입, console 부분 수정, 제거 했습니다.
@JeonDoGyun JeonDoGyun changed the title 등록하기, 수정하기 페이지 모달 안내 텍스트 요청에 따라 적적하게 변경 등록하기, 수정하기 페이지 모달 안내 텍스트 요청에 따라 적절하게 변경 Nov 8, 2023
@JeonDoGyun JeonDoGyun merged commit 9be7d4a into develop Nov 8, 2023
1 check passed
@JeonDoGyun JeonDoGyun deleted the feat/#149 branch November 8, 2023 13:14
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