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

모집의뢰서 중복 예외처리 오류 수정 #98

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jikwan0327
Copy link
Member

What is this PR?🔍

작업내용📑

기존에도 모집의뢰서 중복 작성에 대한 오류처리 코드가 존재하였지만 조건문에 걸리지 않고 있어 다른 메세지를 띄워주고 있던 부분을 올바르게 수정하였습니다

스크린샷📸

공유 사항

Issues

  • close #

Copy link

vercel bot commented Oct 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
jobis ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2024 3:30pm
jobis-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2024 3:30pm

Copy link

nx-cloud bot commented Oct 6, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 7ec45ae. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@phyuna0525
Copy link
Contributor

작성한 뒤 작성완료 버튼을 누르면 에러 처리하는 것이 아닌 모집의뢰서가 있으면 실습 종류 선택페이지에서 비활성화하는것으로 결정했습니다!

@jikwan0327
Copy link
Member Author

작성한 뒤 작성완료 버튼을 누르면 에러 처리하는 것이 아닌 모집의뢰서가 있으면 실습 종류 선택페이지에서 비활성화하는것으로 결정했습니다!

아하 그렇군요. 그래도 일단 사용자가 url을 통해서 비정상적인 접근을 할 가능성도 있기때문에 일단은 머지하는 방향으로 가는게 좋을 거 같습니다

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.

4 participants