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

[SETTING] scraper redux, redux-thunk 세팅 #33

Merged
merged 6 commits into from
Nov 7, 2021
Merged

[SETTING] scraper redux, redux-thunk 세팅 #33

merged 6 commits into from
Nov 7, 2021

Conversation

kingyong9169
Copy link
Contributor

👀 이슈

resolve #27

📌 개요

scraper에서 사용할 redux, redux-thunk를 이용한 api 호출 기본 세팅입니다.

👩‍💻 작업 사항

scraper에서 사용할 action, reducer, store 기본 세팅을 하였고 실행을 통해 순수 redux는 작동한다는 것을 알게 되었습니다.
하지만 redux-thunk를 사용할 때, store.dispatch(success_calender())처럼 dispatch안에 함수(파라미터)를 넣어 api호출하는 것을 예전에 프로젝트(javascript)하면서 했었는데 typescript로 하려니 type맞추는 부분에서 어려운 부분이 있어 pr에 도움요청합니다ㅠㅠ

✅ 참고 사항

스크린샷 2021-10-28 오전 8 49 04

@jaryapp
Copy link
Contributor

jaryapp commented Nov 4, 2021

@kingyong9169
브루니 혹시 이 에러 지금도 발생하나여?
저도 이 에러 났던것 같은데 갑자기 또 안나네요 ㅋㅋ
image

Comment on lines 7 to 19
export function success_calender():ThunkAction<void,RootState,null, CalenderAction>{
return async dispatch =>{
try{
const response = await axios.get(`https://api.github.com/users/kingyong9169`)
dispatch({type: SUCCESS_CALENDER, payload: response.data});
return Promise.resolve(response.data);
} catch(err){
return Promise.reject(err);
}
}
}

export function fail_calender():ThunkAction<void,RootState,null,CalenderAction>{
Copy link
Contributor

Choose a reason for hiding this comment

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

함수명은 camelCase로 변경 부탁드려요~ 참고: CMI-OSS/common#2

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Roger that!

@@ -0,0 +1,29 @@
import { CalenderAction } from './../interfaces/ActionInterfaces/CalenderInterface';
import { RootState } from './../reducers/index';
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
import { RootState } from './../reducers/index';
import { RootState } from './../reducers';

아마 이렇게 하셔도 될거에용(아마?)
기본적으로 index를 보고있어서

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좋습니다~

@kingyong9169
Copy link
Contributor Author

@kingyong9169 브루니 혹시 이 에러 지금도 발생하나여? 저도 이 에러 났던것 같은데 갑자기 또 안나네요 ㅋㅋ image

전 아직 그대로에여ㅠㅠ

@kingyong9169
Copy link
Contributor Author

👀 이슈

resolve #27

📌 개요

scraper에서 사용할 redux, redux-thunk를 이용한 api 호출 기본 세팅입니다.

👩‍💻 작업 사항

scraper에서 사용할 action, reducer, store 기본 세팅을 하였고 실행을 통해 순수 redux는 작동한다는 것을 알게 되었습니다. 하지만 redux-thunk를 사용할 때, store.dispatch(success_calender())처럼 dispatch안에 함수(파라미터)를 넣어 api호출하는 것을 예전에 프로젝트(javascript)하면서 했었는데 typescript로 하려니 type맞추는 부분에서 어려운 부분이 있어 pr에 도움요청합니다ㅠㅠ

✅ 참고 사항

스크린샷 2021-10-28 오전 8 49 04

아래 링크로 해결은 했지만 껄끄러운 부분이 있습니다..
stackoverflow
아래 사진은 store.dispatch (github api 테스트용으로 사용) 후 Promise반환 결과입니다! 정상적으로 작동되는 모습을 확인할 수 있습니다!
스크린샷 2021-11-04 오후 11 30 56

@jaryapp
Copy link
Contributor

jaryapp commented Nov 4, 2021

@kingyong9169
오.. 해결하셨다니 다행이네여 껄끄러운 부분은 어떤 부분인가여?

@kingyong9169
Copy link
Contributor Author

kingyong9169 commented Nov 4, 2021

@kingyong9169 오.. 해결하셨다니 다행이네여 껄끄러운 부분은 어떤 부분인가여?

dispatch옆에 <any>도 붙이고 싶진 않은데.. 뭔가 방법이 있을 것 같은 그런 느낌이라 조금 껄끄럽습니당ㅋㅋㅋㅋ

@jaryapp
Copy link
Contributor

jaryapp commented Nov 7, 2021

@kingyong9169
그러네요 흠.. 일단 머지하고 나중에 개선하는걸로 하죠 ㅋㅋ 그리고 hotfix는 운영중에 발생한 이슈를 처리할때 사용하는 prefix라서 지금은 사용하실 필요없어여 나중에 리얼배포후에 이슈가 발생하면 master에서 브랜치 팔때 사용하는거에요~

@jaryapp jaryapp merged commit ec7f34d into dev Nov 7, 2021
@jaryapp jaryapp deleted the setting/i27 branch November 7, 2021 14:42
@kingyong9169
Copy link
Contributor Author

@kingyong9169 그러네요 흠.. 일단 머지하고 나중에 개선하는걸로 하죠 ㅋㅋ 그리고 hotfix는 운영중에 발생한 이슈를 처리할때 사용하는 prefix라서 지금은 사용하실 필요없어여 나중에 리얼배포후에 이슈가 발생하면 master에서 브랜치 팔때 사용하는거에요~

앗 뭣모르고 그냥 썼군요.. 명심하겠슴당!

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.

[SETTING] scraper redux, redux-thunk 세팅
3 participants