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: 시간표 API mock 서버 연결 해제 및 실제 서버 연결 #25

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

Nangniya
Copy link
Contributor

요약 (Summary)

시간표 API mock 서버 연결 해제 및 실제 서버 연결

변경 사항 (Changes)

시간표 API mock 서버에 연결되어있던 것 실제 서버로 연결

리뷰 요구사항

  • mock 서버가 제한 요청량을 초과하여 더이상 안되어서 실제서버로 바꿨습니다.
  • 근데 실제 서버에는 name건국대학교 3-1학기를 넣으면 404로 응답이 옵니다. @judemin
    • 임시로 건국대학교 3-1학기를 넣고 있었는데 이 부분을 어떻게 해야할지도 정해야 할 것 같습니다.

확인 방법 (선택)

image

Copy link

Visit the preview URL for this PR (updated for commit 7b4df38):

https://hearus-668d7--pr25-fix-api-url-4l34yap8.web.app

(expires Tue, 30 Jul 2024 12:53:10 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: bf5e680be215afe513793a0ffcca015694c306c5

@Nangniya Nangniya merged commit b0e8a1e into develop Jul 23, 2024
2 checks passed
Copy link
Contributor

judemin commented Jul 23, 2024

해당 부분은 아마 실제 서버 URL 로 param을 요청할 때 space가 있어서 제대로 String을 입력하지 못해서 발생하는 문제일 것 같습니다

혹시 axios에서 요청할 때 해당 param을 "" 등으로 묶어서 보내는걸 시도해봐주실 수 있을까요?

@Nangniya
Copy link
Contributor Author

Nangniya commented Jul 23, 2024

해당 부분은 아마 실제 서버 URL 로 param을 요청할 때 space가 있어서 제대로 String을 입력하지 못해서 발생하는 문제일 것 같습니다

혹시 axios에서 요청할 때 해당 param을 "" 등으로 묶어서 보내는걸 시도해봐주실 수 있을까요?

공백을 제거해도 똑같네요 ㅜ 참고로 axios는 안쓰고있습니다.
그리고 mock서버로 했을때는 됐었습니다!

@Nangniya
Copy link
Contributor Author

해당 부분은 아마 실제 서버 URL 로 param을 요청할 때 space가 있어서 제대로 String을 입력하지 못해서 발생하는 문제일 것 같습니다

혹시 axios에서 요청할 때 해당 param을 "" 등으로 묶어서 보내는걸 시도해봐주실 수 있을까요?

image
Postman으로 시도하면 또 Internal Server Error라고 뜨네요..

@Nangniya Nangniya deleted the fix/api-url branch July 23, 2024 13:31
@judemin
Copy link
Contributor

judemin commented Jul 23, 2024

문제를 확인해본 결과
image
위와 같이 현재 구동되고 있는 서버 로그에서 schudule이 비어있다고 나와서
image
위와 같이 DB 확인해보니 데이터가 없어서 발생하는 문제인 것 같습니다

Schedule Element 색상 속성 추가와 함께 조치를 취해야할 것 같은데 내일, 모레 중으로 해당 사항 수정하겠습니다!
아마 지금도 Postman으로 Schedule 추가 -> ScheduleElement 추가하면 동작할겁니다!

TODO : Schedule이 존재하지 않을 경우 예외처리
TODO : Schedule Element 색상 속성 추가

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.

2 participants