React-native-video 문제 그리고 첫 opensource Pr

react-native-video 오픈소스를 사용하며, 가벼운 문제를 발견했다. 이 문제 때문에 기능에 제한이 생겼고 이를 발견한 나는 pr까지 올리는 경험을 하게 되었다.

 

문제

video filter 프로퍼티 값을 넘겨도 제대로 출력이 되지 않음.

 

원인파악

1. debuging

Xcode를 통해 해당 라이브러리 디버깅을 한다.

2. not break

해당 부분을 디버그를 했지만, break가 되지 않는 것을 발견

 

3. 추적

react-native단에서 부터 추적해 나간다

 

4. 문제 발견

react-native-video는 자체적으로 typescript로 되지 않아 @types/react-native-video을 통해 type compile과정을 겪는다. 여기서 문제가 발생했다.

위의 사진을 보면 filterEnable이라는 속성 값이 있다. 왜 여기가 문제인가 바로,,, 오타,,,,

 

RCT_EXPORT_VIEW_PROPERTY부분을 보면, filterEnabled,, d가 빠진 것이다. d를 넣고 실행해보면 정상 실행이 된다.

 

해결

단순히 고치고 해당 부분만 가져다가 쓸 수 있지만, 오픈소스에 기여를 해보기로 결심했다. 

 

1. fork 

https://github.com/DefinitelyTyped/DefinitelyTyped

GitHub - DefinitelyTyped/DefinitelyTyped: The repository for high quality TypeScript type definitions.

 

해당 소스를 포크를 뜬다.

 

2. 브랜치 생성, 수정 그리고 pr

포크를 뜨고, 브랜치 생성 후 해당 부분을 수정 ( 오타 수정 )까지 한 후 pr을 올렸다.

 

 

3. 리뷰 진행

pr test가 오류 나서 관계 없는 원인이라고 하여 다시 pr trigger

 

하지만, 다시 실패,, 그래서 물어봤지만,, 로그 보고 혼자 해결,, ( test 코드도 수정해서 다시 push ) 간단한 문제라서 민망했다.

 

다시 리뷰 후 

 

Ready to merge

아주 짜릿한 경험,, 오픈 소스에 나의 이름이,,

제대로 반영된나 확인,,, 들어갔다 ,,