-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: Apply Mock API to Demo Page #52
Conversation
데모페이지에 Mock API적용하였습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
src/pages/research/Demo.js
Outdated
// { id: 13, title: "데모13" }, | ||
// { id: 14, title: "데모14" }, | ||
// { id: 15, title: "데모15" }, | ||
// ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그냥 주석처리한 더미데이터 지워버리죠. api 어떻게 받아오는지 코드 내에서 확인할 수 있게 지우지 말고 주석으로 남겨달라고 했었는데 데이터 요청 주소 직접 들어가면 확인 가능하네요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
네
- 더미데이터 삭제 - subHeader 수정
참고가 될 것 같아서 국민대학교 데모 사이트 첨부합니다 여기도 표 형식으로 데모가 구현되어 있는데 클릭하면 이런식으로 넘어가는 형태입니다 |
저는 구지 url을 타고 데모 사이트로 이동하지 않아도 홈페이지에서 어떤 역할을 하는 데모인지 설명을 적어 놓는 것이 맞다고 생각하는데 설명을 적는 공간이 적절하지 않다보니 고민이 되네요. 이 부분은 수인님이 판단한 대로 구현합시다.
아 좋네요 알겠습니다 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 확인했습니다!
LGTM! |
확인해본 결과 이러한 에러 코드가 나오긴하는데
toast ui 적용할 때부터 저한테는 나왔던 에러 코드입니다
해결방법은 미리 찾아놨는데요 참고로 해당 에러는 toast ui 공식 깃허브에도 이슈로 등록이 되어 있습니다
|
저도 에러가 뜨네요.
|
윈도우에서는 수정하면 실행되지 않습니다. 따라서 적용하지 않았습니다. |
실행은 되지만 여전히 warning이 뜹니다. |
결론적으로 위 두분히 말씀해주신거 적용해서 해보았는데 |
데모페이지에 Mock API적용하였습니다.