Feature(#17, #19, #20, #21, #205): home 페이지 API 연동, modernizr 적용 #229
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 강의 생성과 강의 조회에 관련된 API 코드를 작성했습니다.
- 강의 생성과 강의 조회에 관련된 API 코드를 작성했습니다.
- set method만 사용하는 useToast에서 recoil method를 교체하였습니다.
- 발표자 페이지 접근을 막기 위해 Modernizr를 적용하였습니다.
- ts type 오류를 해결하였습니다.
- 강의 생성과 강의 조회에 관련된 API 코드를 작성했습니다.
- set method만 사용하는 useToast에서 recoil method를 교체하였습니다.
- 발표자 페이지 접근을 막기 위해 Modernizr를 적용하였습니다.
- 강의 생성과 강의 조회에 관련된 API 코드를 작성했습니다.
…ub.com/LellowMellow/web13_Boarlog into feature/231207-add-lecture-api-at-home
- API 요청과 관련하여 처리를 완료하였습니다.
✅ Deploy Preview for boarlog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
- 배포를 위해 불필요한 코드를 제거하였습니다.
Byeonjin
reviewed
Dec 7, 2023
Comment on lines
+1
to
+9
| declare global { | ||
| interface ModernizrStatic { | ||
| touchevents: boolean; | ||
| } | ||
|
|
||
| var Modernizr: ModernizrStatic; | ||
| } | ||
|
|
||
| export {}; |
There was a problem hiding this comment.
아 이렇게 타입을 처리하는 방법이 있군요.
혹시 문제가 되지 않는다면 var를 const나 let으로 바꿔주는건 어떨까요?
There was a problem hiding this comment.
헉 그렇네요! 바로 반영해보겠습니다 :)
- var를 let으로 변경하였습니다.
Byeonjin
reviewed
Dec 7, 2023
Comment on lines
+18
to
+31
| const handleCreateButtonClicked = () => { | ||
| axios | ||
| .post(`${import.meta.env.VITE_API_SERVER_URL}/lecture`, { | ||
| title, | ||
| description, | ||
| email: "[email protected]" | ||
| }) | ||
| .then((result) => { | ||
| navigate(`/instructor?roomid=${result.data.code}`); | ||
| }) | ||
| .catch((error) => { | ||
| console.log(error); | ||
| }); | ||
| }; |
There was a problem hiding this comment.
아하 이렇게 API 요청을 하는 군요!
배웠습니다
There was a problem hiding this comment.
사실 이 코드는 예전에 제가 사용했던 방법이라 좋은 방식인지는 잘 모르겠어요 ㅠㅠ 나중에 개선할 때 좀 더 생각해봐요 :)
Jw705
approved these changes
Dec 7, 2023
frontend/global.d.ts
Outdated
| interface ModernizrStatic { | ||
| touchevents: boolean; | ||
| } | ||
| var Modernizr: ModernizrStatic; |
There was a problem hiding this comment.
let으로 바꿔도 좋을 것 같습니다!
Suggested change
| var Modernizr: ModernizrStatic; | |
| let Modernizr: ModernizrStatic; |
| @@ -0,0 +1,123 @@ | |||
| /*! modernizr 3.6.0 (Custom Build) | MIT * | |||
Byeonjin
approved these changes
Dec 7, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 개요
작업 사항
고민한 점들(필수 X)
modernizr 처리
상단 페이지에서
touch events를 선택하여modernizr-custom.js를 빌드하여 이를src폴더에 위치시킵니다. 전역으로Modernizr를 사용하기 위하여index.html에script태그를 이용하여 이를 불러옵니다.TypeScript 환경에서 작업하고 있기 때문에 이에 대한 type을 선언하였습니다.
types폴더 하위에 다양한 type들을 정의할 수 있도록 작업하려 하였으나 기존에 작업하였던"types": ["vite-plugin-svgr/client"],과typesRoot설정이 겹쳐svg관련 코드가 정상적으로 동작하지 않을 것을 고려하여global로 관리할 수 있도록 하였습니다.src하위에global.d.ts를 정의하여 아래와 같이type을 정의하였습니다.이를 적용하기 위해
tsconfig.json에 다음을 추가하였습니다.이를 바탕으로
Modernizr는 아래와 같이 활용할 수 있습니다.API 연결
강의 생성과 강의 참여에 관련된 API 연동을 완료하였습니다.
env파일에 추가적인 환경변수를 정의하였습니다.이를 활용하여 각각 API를 연동하였습니다.
스크린샷(필수 X)
2023-12-07.20-57-26.mp4