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

Feature/#369 학습자료 생성 검증 로직 #382

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

Conversation

llddang
Copy link
Collaborator

@llddang llddang commented Jan 12, 2025

관련 이슈

작업 요약

  • 학습자료 생성 시, 학습 파일이 존재하는지 검증 로직 추가했습니다.
  • 한 모달에서 학습자료 생성 버튼을 클릭하면 pending으로 넘어가 pending 상태일때는 버튼을 클릭해도 학습자료가 생성되지 않도록 했습니다.

작업 상세 설명

  • 학습자료 생성 시, 학습 파일이 존재하지않으면 alert가 뜨도록 구현했습니다.

리뷰 요구 사항

  • 제쪽에서 로그인 오류로 제대로된 확인은 못 했는데요. 한번 확인 부탁드립니다.

@llddang llddang self-assigned this Jan 12, 2025
Copy link
Collaborator

@yeonddori yeonddori left a comment

Choose a reason for hiding this comment

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

Pending을 추가한 이유가 중복 클릭 방지인 것으로 보이는데, 제 생각에는 한 번 클릭해서 alert가 뜨더라도, 다시 버튼을 클릭하면 alert로 또다시 알려주는 것이 더 적절할 것 같아요. 혹시 어떻게 생각하시나요? 의도하신 것 같아서 조심스레 여쭈어봅니다!

@llddang
Copy link
Collaborator Author

llddang commented Jan 26, 2025

Pending을 추가한 이유가 중복 클릭 방지인 것으로 보이는데, 제 생각에는 한 번 클릭해서 alert가 뜨더라도, 다시 버튼을 클릭하면 alert로 또다시 알려주는 것이 더 적절할 것 같아요. 혹시 어떻게 생각하시나요? 의도하신 것 같아서 조심스레 여쭈어봅니다!

아 맞습니다!
또한 그 pending을 앞쪽에 할 경우 학습자료를 업로드하고 버튼 클릭시 아무것도 발생하지 않는 에러가 있어서 수정했는데,
그 커밋을 푸시 안했네용ㅋㅋㅋ 지금 해놓겠습니다!

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.

[FEAT] 학습자료 생성에서 검증 로직 구현 및 중복 생성 방지
2 participants