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

fix-fe: 프로세스 설명이 바뀌지 않고 수정을 누르면 400에러 #431

Merged
merged 2 commits into from
Aug 14, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 13, 2024

Original issue description

목적

프로세스 설명이 바뀌지 않고 수정을 누르면 400에러

작업 세부사항

  • 프로세스 설명이 바뀌지 않고 수정을 누르면 수정요청을 보내지 않는다.

참고 사항

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


FIX_PROCESS_DESCRIPTION

closes #430

단순히 수정 버튼을 disabled로 처리했는데, 이게 최선일까용? 다른 좋은 방법은 없을까용? 없으면 말구요~

@github-actions github-actions bot added fix 버그 수정 frontend 프론트엔드 labels Aug 13, 2024
Copy link
Contributor Author

1723601250.773109

Copy link
Contributor

@seongjinme seongjinme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저는 이렇게 입력값의 차이 발생 여부에 따라 명시적으로 "수정" 버튼이 활성화/비활성화 되는 스타일이 더 나은 것 같아요. 불필요한 버튼 액션도 방지할 수 있고요. 특별히 추가드릴 코멘트가 없어 Approve 드립니다. 👍

Copy link
Contributor

@llqqssttyy llqqssttyy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

다른 방법이라면 onSubmit 이벤트에서 내용이 바뀌지 않은 경우 사용자에게 alert을 보이고 return 시키는 방법이 있을 것 같네요.
두 방법을 비교해 보면 사용자가 안내 문구를 읽게 하는 것 보단 비활성화된 버튼을 통해 '제출할 수 없음'을 알려주는 게 더 직관적일 거란 의견입니다~

@lurgi lurgi merged commit c72e41a into fe/develop Aug 14, 2024
28 checks passed
@lurgi lurgi deleted the fe-430-_PROCESS_DESCRIPTION branch August 14, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그 수정 frontend 프론트엔드
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants