react-viewer icon indicating copy to clipboard operation
react-viewer copied to clipboard

[react-viewer] react-redux 6이상 호환 추가

Open HelloWorld017 opened this issue 1 year ago • 2 comments

문제

  • react-viewerwithRef 라는 필드를 사용합니다.
  • react-redux@>=6 부터 withRef 가 없어지며 (forwardRef 로 변경됐습니다.) 사용할 시에는 무려 에러를 내뱉습니다.
    • Interop을 위해 withRefforwardRef 를 모두 명시하는 것이 불가능해졌습니다.

해결

  • Provider.propTypes 가 버전이 올라가며 바뀌기 때문에, 이 사이드이펙트를 통해 버전을 분기하였습니다.
    • 최신 버전에서는 propTypes 가 제거되었고, 6에 context 필드가 추가되었기 때문에 이를 통해 5 이하를 분기할 수 있습니다.
  • react-redux@5 일 시에는 withRef 를 사용, react-redux@>=6 일 시에는 대신에 forwardRef 를 사용하게 변경하였습니다.

HelloWorld017 avatar Sep 18 '23 11:09 HelloWorld017

web-viewer 가 redux5 와 redux6 을 같이 쓰는건가요?

아뇨 react-redux@8 만 써서 사실 redux5 지원을 날려도 상관없긴 합니다. 그러나 혹시 몰라서 일단은 둘 다 지원하게 해뒀습니다. 그냥 떼고 package.json 의 버전을 6으로 올리는게 나을까요..?

HelloWorld017 avatar Sep 21 '23 02:09 HelloWorld017

web-viewer 가 redux5 와 redux6 을 같이 쓰는건가요?

아뇨 react-redux@8 만 써서 사실 redux5 지원을 날려도 상관없긴 합니다. 그러나 혹시 몰라서 일단은 둘 다 지원하게 해뒀습니다. 그냥 떼고 package.json 의 버전을 6으로 올리는게 나을까요..?

보험이라면 들어둬도 될것 같네요. 일단 유지하시죠.

ridi-junhee avatar Sep 25 '23 00:09 ridi-junhee